-
Notifications
You must be signed in to change notification settings - Fork 3
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
geogrid.exe fails due to geog directory named "_con" #43
Comments
Did you download the geographical data via gis4wrf? It specifically handles the "con" case during extraction on Windows. |
@conorsy we do take care of this in out sister project GIS4WRF automatically (here) but we have not documented how to do this manually in the tutorial -- thanks for pointing this out! 👍 In essence after unarchiving geog data folder, you are left with a directory named
where <ABSOLUTE_PATH_TO_FOLDER> is the PATH to the parent directory of
To later remove the folder -- you can then use the command prompt again with the following:
I will try to document this in the tutorial but if manage, would would also appreciate contributions -- The page to amend is at https://github.com/GIS4WRF/gis4wrf.github.io/blob/dmey/tutorials/docs/tutorials/wrf-cmake/simulate-the-2018-european-heat-wave-with-wrf-cmake.md and you can directly edit it on GitHub and create a PR or manually fork it and send us a PR if you fancy. |
@dmey The tutorial should be fixed. It shouldn't advise people to manually download and unzip data but instead use the GUI in gis4wrf, same as what's advised for met data. |
@letmaik the tutorial was originally meant to be used by users not wishing to use GIS4WRF. There is another tutorial for GIS4WRF only at https://gis4wrf.github.io/tutorials/. We can change this but I think it would be an extra burden for people to install and set up QGIS and GIS4WRF if the purpose is to simply run WRF simulations. Let me know what you think but I would simply make a note about this in the current tutorial. |
@conorsy one other thing, since this is really a limitation on Windows due to the current naming used it may be worth raising the concern with the WRF Team directly by opening an issue on https://github.com/wrf-model/WRF or https://github.com/wrf-model/WPS and linking it to this one. |
Wow, thanks both for your speedy replies! I can confirm that the command-line renaming worked on my Windows 10 Pro. I'll see if I can amend the tutorial page accordingly, and will raise an issue on the wrf-model/WPS page. I am interested in using WRF binaries on Windows for teaching purposes. I'll be teaching the students how to do data analysis with python/xarray/cartopy, so am much happier not having to get them to install QGIS and GISWRF. All the best. |
Hi again, I got the reply below from @mgduda in WRF: It seems like simply renaming the con sub-directories in the GEOGRID.TBL file to _con might allow the WPS to handle the renaming that Windows performs:
I think we'd be disinclined to make this change in the repository, but it might be another option for Windows users. So, if you (WRF-CMake) could use a different GEOGRID.TBL when building the binaries for Windows, then that would solve it! (I'm not sure how high a priority this is, though...) |
@conorsy Renaming to |
Fair point. Thanks again for the workaround. |
Describe the bug
geogrid.exe is looking for orography data in DATA\geog/orogwd_1deg/con/index. However, windows does not allow any directory to be named "con". When geog data are unpacked, this directory gets named "_con", and cannot be renamed to "con". Hence, geogrid.exe fails.
To Reproduce
Steps to reproduce the behavior:
Run geogrid.exe following standard test case on https://gis4wrf.github.io/tutorials/wrf-cmake/simulate-the-2018-european-heat-wave-with-wrf-cmake/#download-geographical-data
Expected behavior
Code is actually behaving as expected. The problem is the path geogrid.exe uses to search for orography data, and the inability of Windows to name a directory "con".
System information (please complete the following information):
Additional context
The code runs, so it's not really a bug, but geogrid.exe won't work without the orog data, so it's not really a feature request either...
The text was updated successfully, but these errors were encountered: