-
Notifications
You must be signed in to change notification settings - Fork 0
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
Develop V0 LUH2 data processing tool and workflow #25
Comments
Future tasks
|
I'm seeing a warning about latitude being outside the expected bounds:
That said, the bounds of the variables or the coordinates look fine. Not sure what the issue is here. |
Yep, I saw that too and was also puzzled, but chose to ignore it for now. |
Stumbled into issue pydata/xarray#5581 when attempting to give users the ability to select the time range they want when generating new luh2 output files. Commit f8fd35f updates the minimum version of xarray to incorporate this fix. |
Ran into another dependency issue. Make sure to note the major breaking changes that are noted here: pangeo-data/xESMF#246 |
Hi @glemieux I was looking more at the output data and realize that we have missed a step so far in the data processing workflow. The sum of all states should always equal 1, but in practice it doesn't, because in the LUH2 data they have already multiplied all the states and fluxes by a water&ice fraction dataset first. So the sum of states equals (1 - water&ice fraction). The water & ice fraction data is the file 'staticData_quarterdeg.nc', specifically the variable So I think what we want to do is to also regrid I also haven't fully processed that there are two separate conservative regridding routines in XESMF, "conservative" and "conservative_normed". I am not totally sure I understand the differences fully, but I think we actually want to use "conservative_normed". See https://xesmf.readthedocs.io/en/latest/notebooks/Compare_algorithms.html for some more details. |
We should probably also use the i.e. mask all gridcells where |
ok great. Since it is less than a percent everywhere, I suggest just multiply all states and transitions by a correction term to make the states sum exactly to one, at least until we figure out what the reason for the mismatch is. |
@ckoven the python module and scripts for this tool are complete enough that you should be able to simply run the
From there you should modify the filenames in the
|
Testing on
I think this might be due to the fact that I'll attempt to bring in UPDATE: this looks like it won't be the straight forward option:
I'm attempting to pin my local version of numpy to see if that improves the compatibility issue. I may still need to remove |
I believe the issue here is specific to xesmf version installed on my conda environment, which is a clone of the I'm going to look at the recommended ctsm conda environment creation workflow to see if I need to update the install process. |
Looking at the conda environment creation script for ctsm was pretty straightforward. Realizing that there is a 'latest' version of the condafile, I managed to create an environment by simply modifying this file by adding xesmf. I'm not sure what the minimum necessary version of numpy (or possibly a different package) there is to work with xesmf to avoid this issue. I reviewed issues on the xesmf github repo to see if anyone had noted specific compatibility errors, but nothing popped up. |
It looks like this is a It looks like the underlying issue might be due to what is reported in the numba/numba#7339 (comment). I checked this by spinning up a few
So it looks like the referenced comment isn't necessarily correct, but there are very obviously a bunch of issues with |
Minor changes to landuse_fixes branch
Adapt @ckoven's prototype python code to develop a FATES tool to process LUH2 data.
Tasks
Testing
perlmutter
summit
cheyenne
The text was updated successfully, but these errors were encountered: