-
Notifications
You must be signed in to change notification settings - Fork 20
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
Howto for adding datasets to intake catalog #91
Conversation
934dfb7
to
4f7b640
Compare
@Geet-George @raphvog please have a look a this PR and suggest edits. Maybe you could even try to add your own or favourite missing dataset to the catalog and the how-to page to test the documentations. |
a693f34
to
8a0e2aa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. :) I have never tried the web interface, so I'll leave that to your expertise.
how_to_eurec4a/howto.md
Outdated
|
||
Depending on the complexity of the dataset, an entry can be directly added to the `main.yaml` file of the respective platform/simulation | ||
```bash | ||
vi <platform>/main.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it necessary to specify vim here (and in the following instances)? We could just say the file name and the user can use their means of populating the YAML.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just want to give an example here and vi should be installed on most platforms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly that's true, but I had some recent issues on someone's machine which is why realised this. I think it might be Windows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But I am not particularly hard and fast on this. If this is the only thing remaining, let's go... If there are issues, users would hopefully raise an issue.
c7496c0
to
f1bff4a
Compare
Thanks for the comments @Geet-George! I implemented most of them. Please feel free to also use the |
Ah ja, sorry I forgot |
closes #106 |
In order to make it easier for the community to add datasets to the EUREC4A intake catalog and the HowTo EUREC4A book this PR adds some documentation on how to do so.