Monthly NRT #168
Merged
Monthly NRT #168
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This may belong better in the VM repo, where specifics about how the environment are setup could be included...
|
This does not update any attrs to reflect the NRT nature of this product. Need to update the title/version/summary/metadata_link attrs similar to how the daily nrt code does it. |
sc0tts
reviewed
Sep 27, 2024
| daily-nrt --last-n-days 5 --hemisphere both | ||
| ``` | ||
|
|
||
| Note that the `--overwrite` flag can be used to re-create NRT data if e.g., a |
There was a problem hiding this comment.
Do you want to explicitly give the command to use the overwrite flag?
eg
daily-nrt --last-n-days 5 --hemisphere both --overwrite
...in this README.md ?
sc0tts
approved these changes
Sep 27, 2024
Improves readability. Originally thought I'd break this down so that I could separately create a monthly ds so that the monthly nrt cli could override global attrs. But this code is already nrt-aware (`is_nrt`) and so can handle the overrides itself.
This ensures if we decide to change the NRT platform id later, that we update the summary.
Override G10016-specific attrs in monthly NRT files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a CLI for monthly NRT processing.