Skip to content

Quarto mod2#73

Merged
joelostblom merged 11 commits into
masterfrom
quarto_mod2
Jun 27, 2025
Merged

Quarto mod2#73
joelostblom merged 11 commits into
masterfrom
quarto_mod2

Conversation

@zmx721

@zmx721 zmx721 commented May 13, 2025

Copy link
Copy Markdown
Contributor

@joelostblom Here is the PR for module 2.

  • I have specified quarto=1.6.43 in the environment, because the newest 1.7.30 version breaks render.
  • In module2-02-delimiter.qmd, I needed to install openpyxl in the setup cell, although I have added the package to the environment.
  • There are quite a few links pointing to the DSCI 511 repo. Do you want them to be updated to point to this repo?

@zmx721 zmx721 requested a review from joelostblom May 13, 2025 01:42
@github-actions

github-actions Bot commented May 13, 2025

Copy link
Copy Markdown
PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-06-27 00:15 UTC

@zmx721

zmx721 commented Jun 5, 2025

Copy link
Copy Markdown
Contributor Author

In pandas 2.2.*, aggregation methods no longer exclude non-numeric columns automatically. This affects modules/module2/module2-25-grouping_and_aggregating.qmd the most.

@joelostblom

Copy link
Copy Markdown
Collaborator

I have specified quarto=1.6.43 in the environment, because the newest 1.7.30 version breaks render.

Could you open an issue with the exact error message so we can follow up later?

In pandas 2.2.*, aggregation methods no longer exclude non-numeric columns automatically.

Yes, that is a bit annoying, and I think an important reason to update the slides since students are using 2.2.* in the live exercises and might be confusing why they need to add numeric_only there if it is not in the slides.

@joelostblom

Copy link
Copy Markdown
Collaborator

There are quite a few links pointing to the DSCI 511 repo. Do you want them to be updated to point to this repo?

Can you open an issue about this too? I think there are more important things to do, but this could be a nice future fix if we have time

Comment thread modules/module2/module2-02-delimiter.qmd
@zmx721

zmx721 commented Jun 9, 2025

Copy link
Copy Markdown
Contributor Author

@joelostblom The issue is fixed. Please review.

@joelostblom joelostblom merged commit f9f0fac into master Jun 27, 2025
1 check passed
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