-
Notifications
You must be signed in to change notification settings - Fork 145
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
Updated RDA addresses and dataset names #790
base: main
Are you sure you want to change the base?
Conversation
Did not update ./CHANGELOG.rst ./_build/html/CHANGELOG.html ./_build/html/_sources/CHANGELOG.rst.txt These substitutions were made: sed \ -e "s#glade\/collections#glade\/campaign\/collections#" \ -e "s#edu\/ds\/#edu\/datasets\/ds#g" \ -e "s#ds345.0#d345000#g" \ -e "s#ds199.1#d199001#g" \ -e "s#ds090.0#d090000#g" \ -e "s#ds337.0#d337000#g" \ -e "s#ds285.0#d285000#g" \
Does not look like any html files in this pull request. Do you mean the .rst files? |
Apparently the files in _build/html/... are not under git control. I'm guessing those are exclude intentionally. |
Yes those are files from the local build of the documentation. |
How about for the .rst having one reference file for all the data sets so they only need to be changes in one place (when the rda links change, or change to dois). .. CAM6rean :: https://rda.ucar.edu/datasets/d345000/ .. CAM4rean :: https://rda.ucar.edu/datasets/whatever .. NCEPrda :: https://rda.ucar.edu/datasets/blahblah |
That would tidy some of the files nicely. But most of the files are stream files. |
Add this to conf.py them you can have a guide/references.rst (or whatever you want to call it) included in every file. This means you can use
|
Description:
NCAR's Research Data Archive changed their dataset naming convention,
so all the references in DART became outdated.
The "local" (/glade) versions were also moved to a new location (/glade/campaign) when derecho replaced cheyenne.
Did not update
./CHANGELOG.rst
./_build/html/CHANGELOG.html
./_build/html/_sources/CHANGELOG.rst.txt
These substitutions were made:
sed \
-e "s#glade/collections#glade/campaign/collections#" \
-e "s#edu/ds/#edu/datasets/ds#g" \
-e "s#ds345.0#d345000#g" \
-e "s#ds199.1#d199001#g" \
-e "s#ds090.0#d090000#g" \
-e "s#ds337.0#d337000#g" \
-e "s#ds285.0#d285000#g" \
@hkershaw-brown I updated some .html files too, and would like confirmation that that was
the right thing to do.
@braczka Please check some of the CLM files, but I think that you don't need check all of them.
I did not add DOIs for these datasets, which would be more robust, but more work.
Fixes issue
#755
Types of changes
Documentation changes needed?
Tests
I checked the changes in
./models/POP/shell_scripts/cesm2_1/user_datm.streams.txt.CPLHISTForcing.Solar_template
I did not test any of the testable files.
Checklist for merging
Checklist for release
Testing Datasets