inasafe / inasafe

InaSAFE - QGIS plugin for estimating impact from natural disasters
www.inasafe.org
GNU General Public License v3.0
259 stars 135 forks source link

Building Type Report Table mixing with zero and no data #2318

Closed fredychandra closed 8 years ago

fredychandra commented 9 years ago

Problem

The Building Type Report Table still reporting zero (o) and No data if run IF with aggregation. The implication is the report become too long and truncation in the lateral side. The table still use OSM preset/attribute although the exposure data that being used is not come from OSM downloader.

image

image

image

Detail

Test on Dock IF : Earthquake with Building Point Hazard Data : EQ_TTU (Polygon) Exposure : Infrastuvture_BIG (Point) Aggregation : Yes (Village Level)

proposed solution

  1. No need report the zero (0) result. This can save space
  2. The Detail Building Type report should use existing exposure that run in IF

    CC

@ismailsunni @samnawi @Charlotte-Morgan

Charlotte-Morgan commented 9 years ago

see #2320

Charlotte-Morgan commented 9 years ago

Hi @fredychandra i agree - if there is a total of 0 for any type the type can be removed and if all values in a table are 0 or nodata - do not write the table

we probably also need to add a note that tells users why tables are not included

cchristelis commented 9 years ago

Hi @fredychandra, i am unfortunately unable to reproduce with the test data. Is there a way to get the exact scenario to me? Perhaps even just a representative clip of the data?

cchristelis commented 9 years ago

Yeah, the post processors seem to be written with only OSM building types in mind: self.fields_values = OrderedDict([ ('Medical', ['Clinic/Doctor', 'Hospital']), ('Schools', ['School', 'University/College', ]), ('Places of worship', ['Place of Worship - Unitarian', 'Place of Worship - Islam', 'Place of Worship - Buddhist', 'Place of Worship']), ('Residential', ['Residential']), ('Government', ['Government']), ('Public Building', ['Public Building']), ('Fire Station', ['Fire Station']), ('Police Station', ['Police Station']), ('Supermarket', ['Supermarket']), ('Commercial', ['Commercial']), ('Industrial', ['Industrial']), ('Utility', ['Utility']), ('Sports Facility', ['Sports Facility']), ('Other', [])])

fredychandra commented 9 years ago

HI @cchristelis i used these data :

  1. TTU_Earthquake.shp (Polygon) as Hazard
  2. TTU_Infrastucture_BIG.shp as exposure (Non OSM data set)
  3. TTU_Villagge_BIG.shp as aggregation layer

Please find the data that I have used for the test (IF : EQ Polygon with Point with Aggregation) on the link below.

https://drive.google.com/drive/folders/0B_gC8poAYEtefmxrMnRZdGQ5bl8xeHhVVlE4Y2J6T3N4aXVCMl9PcEFjbXZIb2lZQzBmNEU

cchristelis commented 9 years ago

The no data / zero column removal has been added in #2345. The underlying issue is caused by non- osm data, which is not handled in the post processors.

cchristelis commented 9 years ago

This issue should be moved to #2332.

Charlotte-Morgan commented 9 years ago

this relates to non OSM data - lets look at this for 3.3 Milestone and label updated

ismailsunni commented 8 years ago

hi @fredychandra Can you re-share your data above? I try to find it, but the link is no longer valid.

Thanks

felix-yew commented 8 years ago

Hi @ismailsunni we have compiled a check list about Zero and No Data, too, just see in the Zero and No Data (ZND) column. Here is the link

https://docs.google.com/spreadsheets/d/1Cx7XWtK-GoIiGwlZzrcaVxiopwXieX_DKRcwETfq4_Y/edit?usp=sharing

The information about the Impact Function Table of Sequence Order: "V" it means the sequence may appear in map legend and/or table analysis, and needs fixing. "-" it means already good.

Make it happy :smile:

Cc: @Charlotte-Morgan @timlinux @samnawi @ivanbusthomi

timlinux commented 8 years ago

This should be fixed with the value mapping. @ismailsunni please confirm and close

ismailsunni commented 8 years ago

Closed. Fixed in #2904 by @Gustry