-
Notifications
You must be signed in to change notification settings - Fork 23
Update data access section #547
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
Conversation
for more information, see https://pre-commit.ci
👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. |
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.
The text looks great! I put in some suggestions for formatting fixes.
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 great, @ktyle. Thanks!
BR suggested change 1 Co-authored-by: Brian Rose <brose@albany.edu>
BR suggested change 2 Co-authored-by: Brian Rose <brose@albany.edu>
The formatting is still off for section A part 4 ... |
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.
Alternatively, if we don't like the formatting of the definition list, we could just use nested bullets under item 4.
portal/cookbook-guide.md
Outdated
For many concepts, we encourage writing self-contained functions to generate simple representative datasets for demonstrating scientific concepts. Your Cookbook can even reuse these sample data repeatedly throughout. | ||
|
||
4. **For complex Cookbooks that rely on large datasets that are not already accessible through other services**, we suggest two options: | ||
Institutional Repositories |
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 think adding a line break will fix the formatting:
Institutional Repositories | |
Institutional Repositories |
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 great now, thanks @ktyle
This PR updates the Data Access section in the Cookbook Contributor's guide. It can be used as a reference for the upcoming 2025 Summer Cook-off, as well as for any prospective Cookbook contributors.