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

consolidate tag categories #72

Merged
merged 3 commits into from
Jun 26, 2024
Merged

consolidate tag categories #72

merged 3 commits into from
Jun 26, 2024

Conversation

simonpcouch
Copy link
Collaborator

My goal here was to make "categories" in the Learn listing more navigable by halving the number of entries. I mostly did so by 1) removing categories that only have one listing (especially if the category is in the title of the article) and 2) aligning synonyms for the same category, like "tune," "tuning," and "model tuning."

I generally didn't prefer package names over keywords or vice versa—that may be a discussion worth having for a future PR.

Copy link

netlify bot commented Jun 25, 2024

Deploy Preview for tidymodels-org ready!

Name Link
🔨 Latest commit e4a00e4
🔍 Latest deploy log https://app.netlify.com/sites/tidymodels-org/deploys/667c912f9f9a0d0008d61145
😎 Deploy Preview https://deploy-preview-72--tidymodels-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@simonpcouch
Copy link
Collaborator Author

Expected to likely see an error before rendering, but was surprised to see this specifically:

ERROR: Error executing 'Rscript': No such file or directory (os error 2)

Unable to locate an installed version of R.

Any intuition here before I start debugging, @EmilHvitfeldt?

@EmilHvitfeldt
Copy link
Member

Appears to be this issue: quarto-dev/netlify-plugin-quarto#2

@EmilHvitfeldt
Copy link
Member

but if you look at the netlify logs, it doesn't break right away, the first breakage is at learn/models/calibration, so to fix this "easy" we just have to update the _freeze for the breaking pages/the pages you updated

@EmilHvitfeldt
Copy link
Member

works now 🎉

@EmilHvitfeldt EmilHvitfeldt merged commit ab84f6d into main Jun 26, 2024
4 checks passed
@EmilHvitfeldt EmilHvitfeldt deleted the categories branch June 26, 2024 22:11
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