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

Download docs #166

Merged
merged 8 commits into from
Dec 19, 2024
Merged

Download docs #166

merged 8 commits into from
Dec 19, 2024

Conversation

znichollscr
Copy link
Collaborator

@znichollscr znichollscr commented Dec 18, 2024

Description

Closes #118

Checklist

Please confirm that this pull request has done the following:

  • Documentation added (where applicable)
  • Changelog item added to changelog/

@znichollscr
Copy link
Collaborator Author

@durack1 I've started here. Feel free to push directly/comment. I'll finish off tomorrow

@durack1
Copy link
Contributor

durack1 commented Dec 18, 2024

@cecilehannay just pinging you into this PR/thread, so we can make sure that the documentation that we are building answers your questions.. Please feel free to peek at the start - 118-download-examples/docs/usage-data-user/index.md - and see if this is getting close to helping you out

@znichollscr znichollscr requested a review from durack1 December 18, 2024 20:30
@cecilehannay
Copy link

Thanks, I will look at this,

@durack1
Copy link
Contributor

durack1 commented Dec 19, 2024

@znichollscr just added some nits - this is good enough for a first pass, I suggest we merge, and if @cecilehannay has additional queries, we can augment - or feel free to suggest additional additions @cecilehannay!

Copy link
Contributor

@durack1 durack1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, start

@znichollscr znichollscr marked this pull request as ready for review December 19, 2024 06:38
Copy link

No changes to the database between 'main' branch and 4f4832d

Copy link

No changes to the database between 'main' branch and 3ac8b4e

@znichollscr znichollscr merged commit 302a8c7 into main Dec 19, 2024
6 checks passed
@znichollscr znichollscr deleted the 118-download-examples branch December 19, 2024 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document download scripts for models
3 participants