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

File paths, organization and notebooks #35

Merged
merged 9 commits into from
Dec 20, 2024
Merged

Conversation

cansavvy
Copy link
Contributor

@cansavvy cansavvy commented Nov 18, 2024

Purpose/implementation Section

What changes are being implemented in this Pull Request?

Much of this is remixed from other places.

The content is largely here but I will need to review some more.

What was your approach?

Borrowed some stuff from DataTrail and ITN courses but then am tailoring it.

For Quarto/Markdown I describe the Pros and Cons of each, but then keep the markdown as the tutorial and point folks to Quarto should they be interested.

We could also add a section about quarto rendering but I'm hesitant to dive to much into the details because of maintenance concerns. I would rather point people to posit resources for these tutorials and focus in this course on the reproducibility aspects.

Copy link
Contributor

github-actions bot commented Nov 18, 2024

No spelling errors! 🎉
Comment updated at 2024-12-19-15:31:31 with changes from b2d31cd

Copy link
Contributor

github-actions bot commented Nov 18, 2024

No broken url errors! 🎉
Comment updated at 2024-12-19-15:31:30 with changes from b2d31cd

Copy link
Contributor

github-actions bot commented Nov 18, 2024

No quiz formatting errors! 🎉
Comment updated at 2024-12-19-15:31:31 with changes from b2d31cd

Copy link
Contributor

github-actions bot commented Nov 18, 2024

Re-rendered previews from the latest commit:

* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea.

Updated at 2024-12-19 with changes from the latest commit b2d31cd

@cansavvy cansavvy changed the title File paths and organization draft File paths, organization and notebooks Nov 20, 2024
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
- Is easy for you and others to find the files you need quickly [@Shapiro2021].
- Minimizes the likelihood for errors (like writing over files accidentally) [@Shapiro2021].
- Is something maintainable [@Shapiro2021]!

Copy link
Contributor

Choose a reason for hiding this comment

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

idk why but these resources all have question marks in the render preview

Copy link
Contributor Author

Choose a reason for hiding this comment

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

AH shoot I must be missing the references in the bib file. Will look into this.

05-setting-up.Rmd Outdated Show resolved Hide resolved
```

READMEs stick out in a project and are generally universal signal for new people to the project to start by READing them. GitHub automatically will preview your file called "README.md" when someone comes to the main page of your repository which further encourages people looking at your project to read the information in your README.

**Information that should be included in a README:**
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe include who is involved in the project or who to contact? (I imagine some people will not be on GitHub)

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should talk about licenses?

Copy link
Contributor

Choose a reason for hiding this comment

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

Could do it briefly

Copy link
Contributor

Choose a reason for hiding this comment

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

added something below!

05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
- **Put source scripts and functions in their own directory**. Things that should never need to be called directly by yourself or anyone else.
- **Put output in its own directories** like `results` and `plots`.
- **Have a central document (like a README)** that describes the basic information about the analysis and how to re-run it.
- Make it easy on yourself, **dates aren't necessary**. The computer keeps track of those.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe need a bit more info about this... that dates of file updates aren't necessary? if a survey is from 2020 that might be necessary

Copy link
Contributor

Choose a reason for hiding this comment

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

see below


- **Make file names informative** to those who don't have knowledge of the project but avoid using spaces, quotes, or unusual characters in your filenames and folders -- these only serve to make reading in files a nightmare in some programs.
- **Number scripts** in the order that they are run.
- **Keep like-files together** in their own directory: results tables with other results tables, etc. _Including most importantly keeping raw data separate from processed data or other results!_
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe could add sub listing for raw data separate from processed data so we can say a bit more about why? (that it helps us keep from overwriting our raw data)

05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
05-setting-up.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
06-rmarkdown.Rmd Outdated Show resolved Hide resolved
Copy link
Contributor

@carriewright11 carriewright11 left a comment

Choose a reason for hiding this comment

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

OK I read through everything now @cansavvy ! I had some more complicated suggestions with images about paths, happy to discuss these. Looks like we still need to update the bib file too so putting a reminder here :)

@cansavvy cansavvy mentioned this pull request Dec 20, 2024
@cansavvy
Copy link
Contributor Author

OK I read through everything now @cansavvy ! I had some more complicated suggestions with images about paths, happy to discuss these. Looks like we still need to update the bib file too so putting a reminder here :)

Created an issue about the bib file #37 Going to merge this and then work on the bib file updating in a separate PR to keep this PR from getting too big.

@cansavvy cansavvy merged commit eefeec4 into main Dec 20, 2024
11 checks passed
@cansavvy cansavvy deleted the cansavvy/setting-up branch December 20, 2024 14:07
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.

2 participants