-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Is LCZ E highly over estimated ? #935
Comments
Yes sure. Because there is no information to qualify those areas. |
thank you @ebocher ! |
A good way to identify the most probable error cases is to use the UNDEFINED_FRACTION indicator. When it is high the classification becomes very uncertain |
thank you @j3r3m1 : indeed, the value is 0,7446393169013416 But I don't even understand why is this RSU so big. There is also buildings on it (see screenshot - Zoom) |
Wow ! Is it a single one on the image you have provided above ? |
It is, not only on the zoom I provided above but the full black area below is one piece only ! And yes, from a geographic point of view, it's a total non-sense. I'm really surprised because there is no such inconsistency on others neighbors municipalities on the seashore For your information he resulting file (only Ploemeur - 56162) is available at https://drop.chapril.org/download/96cdd49e335325ea/#quY2ZDvU_qcWyiNnwEapkA Just ask me if you want the whole dataset for the given municipality |
OK as strange as I would have imagined. It would be interesting to see if the problem was already there before this new partitionning (I would guess yes). Do you have a version of GeoClimate right before this modification that you could use ? |
I've tried using the given geoclimate snapshot, but whatever the BD-TOPO (2.2, 3.0, 3.3) version I used I got an error.
|
Do you really need the UTRF ? What if you remove it from the indicatorUse list ? |
I suppose the OSM is with the last version, not the one before the RSU modifications ? |
of course ! |
Means there is definitely a good reason for having scripts merging several datasets to take advantage of all types |
I will use OSM data regarding this specific municipality right now. I'm (almost) fine with that 👍 I know that all the RSU approach rely on data but but do you think the algorithm could be enhanced to at least prevent these big and inconsistent units when using BD TOPO ? This would simplify manual handling |
I have no doubt regarding this ! ;-)) |
These big and unconsistent units are unfortunately a problem even for OSM sometimes (#930). I do not think that "simple solution" as said in this other conversation might solve the problem in BDTopo but hopefully most of the OSM cases. That's why a mix of OSM and BDT data might be a good compromise |
I'm working with geoclimate v1.0.0 and BD-TOPO v3.
And I've got the impression that LCZ-E (bare rock or paved) became over-estimated. I can even see LCZ-E covering agricultural fields. Here some samples :
In Ploemeur (56162) - near Lorient :
In Ploemeur (56162) - on the coast part :
The text was updated successfully, but these errors were encountered: