Skip to content
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

Error in NEON surface datasets for agricultural sites #1871

Closed
3 tasks done
wwieder opened this issue Oct 14, 2022 · 2 comments · Fixed by #1872
Closed
3 tasks done

Error in NEON surface datasets for agricultural sites #1871

wwieder opened this issue Oct 14, 2022 · 2 comments · Fixed by #1872
Assignees
Labels
bug something is working incorrectly priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations science Enhancement to or bug impacting science

Comments

@wwieder
Copy link
Contributor

wwieder commented Oct 14, 2022

Brief summary of bug

Subset data wasn't used correctly for agricultural NEON sites (STER and KONA), where PCT_CFT is not 100% rainfed spring wheat, as intended.

ncdump -v PCT_CFT /glade/p/cesmdata/cseg/inputdata/lnd/clm2/surfdata_map/NEON/surfdata_hist_78pfts_CMIP6_simyr2000_KONA_c210831.nc

 PCT_CFT =
  0,
  0,
  27.498169756034,
  0,
  12.2062509947551,
...

This requires using the --crop flag in ./subset_data, as below
./subset_data point --lat 39.10828 --lon -96.61044 --site KONA --dompft 19 --crop --create-surface

** One clarification, @ekluzek is this correct for rainfed spring wheat? **See Table 26.1 in the tech note.

which is subsequently modified for soil properties
./modify_singlept_site_neon.py --neon_site KONA

Updated files are available /glade/scratch/wwieder/single_point_neon_updated/, which seems to be handled correctly

ncdump -v PCT_CFT /glade/scratch/wwieder/single_point_neon_updated/surfdata_0.9x1.25_hist_78pfts_CMIP6_simyr2000_STER_c221014.nc

 PCT_CFT =
  0,
  0,
  0,
  0,
  100,
  0,

...
  • If this all looks correct, @ekluzek can these be moved to inputdata?
  • Subsequently, will the run_neon script automatically point to these latest files?
  • Finally we'll also have to modify this information in the neon_sites file
@wwieder wwieder added priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations bug something is working incorrectly tag: bug - impacts science labels Oct 14, 2022
@wwieder wwieder added this to the NEON milestone Oct 14, 2022
@wwieder
Copy link
Contributor Author

wwieder commented Oct 14, 2022

These files are also not working as intended because PCT_NAT_PFT > 100 with modifications made to bare ground, but not other pfts. In the 'old' surface data these values have all been zeroed out. I'll do that manually now, using

This was caused by an error in modify_singlept_site_neon.py that was manually overwriting
PCT_NAT_PFT.values[0] = [[100.0]], which I don't think is necessary.

  • We need to fix modify_singlept_site_neon.py so this is handled this correctly.

wwieder added a commit to wwieder/ctsm that referenced this issue Oct 14, 2022
wwieder added a commit to wwieder/ctsm that referenced this issue Oct 14, 2022
@wwieder
Copy link
Contributor Author

wwieder commented Oct 14, 2022

@ekluzek new surface data are available here /glade/scratch/wwieder/single_point_neon_updated

I'll open a PR for the code modifications

@wwieder wwieder mentioned this issue Oct 14, 2022
@ekluzek ekluzek removed this from the NEON milestone Apr 25, 2024
@samsrabin samsrabin added the science Enhancement to or bug impacting science label Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations science Enhancement to or bug impacting science
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants