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

submitting data for repro-med-disparities #807

Merged
merged 13 commits into from
Feb 23, 2025

Conversation

katcorr
Copy link
Contributor

@katcorr katcorr commented Feb 6, 2025

Thank you for your submission! Please leave this text as-is when you submit, and then check the checklist.

Usability:

  • This dataset has not already been used in TidyTuesday.
  • The dataset is less than 20MB when saved as a tidy CSV.
  • I can imagine at least one data visualization related to this dataset.

Preparation:

  • cleaning.R has code to download and clean the dataset, resulting in descriptively-named data.frames.
  • I ran saving.R to create csvs and md data dictionaries for each of my data.frames.
  • I filled in descriptions for each variable in each of the {dataset}.md files.
  • I edited the intro.md file to introduce my dataset.
  • I included at least one image for my dataset as a png file.
  • I completed the information in meta.yaml, including descriptive alt text for each image.
  • I provided information in meta.yaml about how to credit me, and deleted any parts of that block that I do not want you to use.

Copy link
Collaborator

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

Thank you for this submission! It looks really interesting! I just have a few comments. I recommend this process:

  1. Read the comments here on GitHub, and "Commit" any of them that are simple edits via the web UI. If you have more changes to make via your local version, follow the next steps.
  2. "Pull" the latest version of this data back to your machine. In RStudio, you can click the "Pull" button in the GitHub pane.
  3. Make any additional changes.
  4. "Push" the changes back to us (the "Push" button on the GitHub pane in RStudio).

Let me know if you have any questions!

Copy link
Collaborator

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

Looks good! There will be a couple more emails as I apply my last edit and then run the acceptance script, so don't be confused by those. Thanks for the submission!

Copy link
Collaborator

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

Marking the point that I've viewed.

@jonthegeek
Copy link
Collaborator

/assign repro-med-disparities 2025-02-25

Copy link
Collaborator

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

Have to make some tweaks to get the script to run.

@jonthegeek
Copy link
Collaborator

/assign repro-med-disparities 2025-02-25

Copy link
Collaborator

@jonthegeek jonthegeek left a comment

Choose a reason for hiding this comment

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

Thanks again!

@jonthegeek jonthegeek merged commit b44b95b into rfordatascience:main Feb 23, 2025
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