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

Update LULC file for v3 historical compset #6339

Merged
merged 1 commit into from
May 30, 2024

Conversation

wlin7
Copy link
Contributor

@wlin7 wlin7 commented Apr 10, 2024

The is to replace the old r05 landuse.timeseries file for the historical compsets
(e.g., F20TR and WCYCL20TR), which is considered outdated (probably using CMIP5 raw data).
This was noticed after the creation of v3.0.0 tag and before the start of v3.LR.historical simulations.
The updated file is currently specified in the run script to override the default.

[non-BFB] for F20TR and WCYCL20TR tests
[NML] same as above for variable flanduse_timeseries

Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6339/
on branch gh-pages at 2024-04-10 17:31 UTC

@rljacob rljacob added this to the v3.0.1 milestone Apr 10, 2024
@rljacob rljacob assigned bishtgautam and unassigned rljacob Apr 11, 2024
@rljacob
Copy link
Member

rljacob commented May 2, 2024

@bishtgautam please review.

@rljacob rljacob requested a review from bbye May 2, 2024 17:49
@bishtgautam
Copy link
Contributor

I moved the following @wlin7's notes from the first comment to here because the first comment
with be copied/pasted in the git commit message.


Notes:

This was noticed after the creation of v3.0.0 tag and before the start of v3.LR.historical simulations.
The updated file is currently specified in the run script to override the default.

@rljacob
Copy link
Member

rljacob commented May 4, 2024

That note is important enough to be in the commit message. PR description edited.

@rljacob
Copy link
Member

rljacob commented May 9, 2024

@bishtgautam you can start merging this

@rljacob
Copy link
Member

rljacob commented May 22, 2024

@bishtgautam any reason not to merge this?

@bishtgautam
Copy link
Contributor

Sorry for being late on merging this. This is ready to go.

@rljacob
Copy link
Member

rljacob commented May 29, 2024

@bishtgautam please start merging it.

bishtgautam added a commit that referenced this pull request May 29, 2024
The is to replace the old r05 landuse.timeseries file for the historical compsets
(e.g., F20TR and WCYCL20TR), which is considered outdated (probably using CMIP5 raw data).
This was noticed after the creation of v3.0.0 tag and before the start of v3.LR.historical simulations.
The updated file is currently specified in the run script to override the default.

[non-BFB] for F20TR and WCYCL20TR tests
[NML] same as above for variable flanduse_timeseries
@bishtgautam
Copy link
Contributor

merged to next.

@bishtgautam bishtgautam merged commit 4aaf158 into master May 30, 2024
11 checks passed
@bishtgautam bishtgautam deleted the wlin/lnd/update_v3hist_lulc branch May 30, 2024 19:33
@rljacob
Copy link
Member

rljacob commented Oct 22, 2024

@wlin7 can you create an issue for this since it was a bug in 3.0.0 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants