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

Changes in dataset #688

Closed
wants to merge 9 commits into from
Closed

Conversation

stacyrse
Copy link

@stacyrse stacyrse commented Nov 1, 2024

  • added a new dataset
  • created new plots
  • created new questions/adapted the old material

This is not the final version, I am hoping to check the layout with the new build

Copy link

github-actions bot commented Nov 1, 2024

⚠️ WARNING ⚠️

This pull request contains a mix of workflow files and regular files. This could be malicious. No preview will be created.

regular files:

  • README.md
  • episodes/05-coffee.md
  • episodes/07-reading-tabular.md
  • episodes/08-data-frames.md
  • episodes/09-plotting.md
  • episodes/10-lunch.md
  • episodes/15-coffee.md
  • episodes/data/data-breast-cancer.csv
  • episodes/data/data-palmers-penguins.csv
  • episodes/fig/basic_plot.png
  • episodes/fig/boxplot.png
  • episodes/fig/heatmap.png
  • episodes/fig/histogram1.png
  • episodes/fig/histogram2.png
  • episodes/fig/lineplot.png
  • episodes/fig/pairplot.png
  • episodes/fig/scatter1.png
  • episodes/fig/scatter2.png
  • episodes/fig/scatter_GG.png
  • episodes/fig/violinplot.png

workflow files:

  • .github/workflows/README.md

@alee
Copy link
Member

alee commented Nov 4, 2024

Hi there, did you submit this PR to the correct base repository? Please make sure you follow the contributor guidelines when submitting a PR, including a clear description of what you're attempting to modify and why.

If you were trying to customize your own research group's version of the Python gapminder lesson you'll want to adjust your PR base repository targets 😅

You should also be careful not to commit generated files, e.g., the figures...

Cheers!

@alee alee closed this Nov 4, 2024
Copy link
Member

@alee alee left a comment

Choose a reason for hiding this comment

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

Appears to be a test and includes generated files that should not be stored in the repo.

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.

3 participants