Skip to content

split up make_function_list.R#148

Open
EmilHvitfeldt wants to merge 10 commits intomainfrom
make_functions_list-refactor
Open

split up make_function_list.R#148
EmilHvitfeldt wants to merge 10 commits intomainfrom
make_functions_list-refactor

Conversation

@EmilHvitfeldt
Copy link
Copy Markdown
Member

@EmilHvitfeldt EmilHvitfeldt commented Mar 10, 2026

This PR refactors make_function_lists.R script into one script for each table corresponding to the tables seen on https://www.tidymodels.org/find/.

  • Split make_function_lists.R into separate scripts in make_function_lists/:
    • _utils.R - shared utilities and caching logic
    • broom.R, recipes.R, tidymodels.R, parsnip.R, sparse.R, tidyclust.R - one for each table
    • run_all.R - runs all of them (same as running make_function_list.R)

I replace pkgdown::as_pkgdown() with direct Rd file parsing via tools::parse_Rd().

The more important change is the implementation of caching. We now store a cache of the things we are fetching in .pkg_cache.csv. Reducing the total time taken to run all of this down to around 3 minutes.
Notice we are still fetching things from websites, so this is still not static.

@EmilHvitfeldt EmilHvitfeldt marked this pull request as ready for review March 10, 2026 23:15
@EmilHvitfeldt
Copy link
Copy Markdown
Member Author

I don't know if this is too much or not. It increases the complexity

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.

1 participant