-
Notifications
You must be signed in to change notification settings - Fork 17
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
Urban Wind Field: URock v2023a - Missing Data in Raster File #704
Comments
I have the same issue, but only on my windows machine. @AlexandraStankulova , are you on windows also? I am simulating on 4 meter resolution, both in horizontal and vertical dimensions, then I have a 2 meter raster grid that I want to use for raster output. I have QGIS 3.40, UMEP 2.1.3. @j3r3m1 , how do you interpolate? Below. you find my data and settings:
|
Yes, I also work with Windows, I have Windows 10. |
The interpolation depends on the resolution of your calculation and the resolution of the raster used as output constraint. However, most of the time (and this is the case for your data) this function is used: And then at the end the no data values are filled using the few lines after this one: I have not faced any issue with Ubuntu for your case @biglimp and I will try the @AlexandraStankulova case. The issue probably comes from one or the other of the code provided above. I now have a Windows OS at home. It may help to fix Windows issues but I need to find time for that. |
@AlexandraStankulova can you please share a smaller file ? Your is more than 1Gb. |
Yes sure. I will upload now only the input data and the output raster windspeed file that actually results with no data. Here the files: |
Interesting, I could reproduce the error. I will investigate what is the issue |
Well the issue was that I did not use the right DSM for interpolating the results at the end... Thus I cannot reproduce the error on Ubuntu. I have to debug on a Windows machine, not sure when. |
I am encountering the same error on my side regarding outputs without data when running the process on Ubuntu. @j3r3m1 Here is a sample to reproduce the error. Let me know if you need any additional information. Thanks in advance for your help! |
OK there was quite a big problem. When the vegetation was not intersected at all by building wake zones (which is often the case in rural area obviously...), there vegetation was removed from the analysis... It is now solved. |
@lgzdbf it should solve your issue. |
Hello. I guess it will solve it definetely because I was doing some different tests, and I noticed that sometimes when I increase the buffer of the data to be inserted also around the area of analysis, sometimes this problem was not appearing (but I coulnd't explain nmyself why with e certain buffer are it works and with another buffer area it doesnt). |
OK the TIN interpolation does not work in some specific cases. I have reported the bug in the QGIS repo. We will see how they can help (qgis/QGIS#60416). Two alternatives can be used:
@biglimp what do you think ? |
@biglimp @lgzdbf @AlexandraStankulova you can already try the first proposition as it it now in the last UMEP version. |
Thanks for your information @j3r3m1. I ran it again for a sample and it works without a problem. I will run it on several tiles to see what happens and let you know. |
IDW might smooth out the result, but since you have lots of data points it should not be an issue. |
@j3r3m1, thank you for the information. I will retry to run the tile that I was analysing with the updated version from the repository and I will let you know. |
Have you tried the workaround proposed in my comment qgis/QGIS#60416 (comment)? |
I confess that after my first reading I only thought that the comment was a confirmation of the error. But actually this is both a workaround that solves the problem but which also speeds up the TIN algorithm... Thanks a lot for your proposition. This is now solved in last UMEP version (2.1.10) |
In version 2.1.10, the simulations seem to run indefinitely (two runs have been running since yesterday at 9 PM). This was not the case in V2.1.9. No errors are reported, but the algorithm appears to run endlessly. The raster WS file is saved within a few minutes after launch, while the VWS and HWS files, as well as the rockle zones, are still not generated. |
Can you please share the settings of the simulation that does not end ? |
Here are the settings for a tile and a day-time that doesn't work. But I don't think you will find anything like this because it's variable. When I rerun the algorithm, some runs that previously didn't finish are processed in just a few minutes. `InputsBUILDINGS: /workdir/hmcis/lagelout/5_application/To_workdir/spatial_input_data/va/grenoble/c-1/vector_build_c-1.fgb|layername=vector_build_c-1 |
OK the problem is probably raised by the fact that the points are randomly sorted before the TIN interpolation. In some situations the random order does not work but works for other orders. I have been trying to implement a way to reapply a new random sorting if the interpolation lasts too long but without success. |
Unfortunately, it looks like my workaround doesn't always workaround the issue... |
Concerning version 2.1.10, the problem might come from my side. I have made a slight modification at this branch (https://github.com/j3r3m1/UMEP-processing/tree/TIN_test - it is named version 2.1.11). Let us know. |
Thank you for your feedback. Still the same issue with the version 2.1.11 test : I will therefore try with IDW to see the result. |
All runs with IDW completed without errors. It is indeed faster with TIN than with IDW, averaging 1 minute compared to 7-8 minutes. However, the result template is different, which is expected. Would you like me to test on a larger sample, or would you prefer to wait and decide if TIN will be kept as the official version? |
Is the "Order by expression" output a GeoPackage layer in your processing algorithm? If yes, then features are not actually reordered. See qgis/QGIS#60432. |
What do you mean by the result template is different ? Can you share screenshots with same value ranges for two simulations using a different version of the plugin (test and main) ?
All tiles have the same behavior or is it a random effects with most of them going up to the end ? |
Yes the output is a gpkg file. But it is resulting in the creation of the primary indexes and a randomization of the order of the point (see below, left is the output gpkg, right is the initial fgb file): Note that if I output a fgb file, there is no primary index created and the rows are still in the same initial order. |
OK and how often do you get the problem for this tile ? Like 1 / 10 ? |
On a tile, over 48 runs, there are about 3-5 problematic runs on average, each time different. |
OK. The "Order by expression" issue only occurs if both input and output layers are GeoPackages or if they have a fid/primary key. |
I am performing the wind analysis using the "Urban Wind Field: URock v2023a". The area is quite large and complex, but after 3hours the analysis finishes with success, loading as always the created raster file with the wind speed.
The problem is that the raster resullts with no data, while the vector file has the vaues in it.
What could be the problem?
Here I upload the input data and the output data created by UMEP:
https://we.tl/t-TZrhbqXpdl
The text was updated successfully, but these errors were encountered: