PALEOtoolkit / PALEOboxes.jl

Model coupler for the PALEO model framework
https://PALEOtoolkit.github.io/PALEOboxes.jl
MIT License
2 stars 1 forks source link

Migrate from SnoopPrecompile to PrecompileTools #76

Closed timholy closed 1 year ago

timholy commented 1 year ago

This pull request migrates the package from SnoopPrecompile to PrecompileTools. PrecompileTools is nearly a drop-in replacement except that there are changes in naming and how developers locally disable precompilation (to make their development workflow more efficient). These changes are described in PrecompileTool's enhanced documentation, which also includes instructions for users on how to set up custom "Startup" packages, handling precompilation tasks that are not amenable to workloads, and tips for troubleshooting.

Why the new package? It meets several goals:

For more information and discussion, see this discourse post.

timholy commented 1 year ago

Not quite sure why it didn't work, maybe change to PrecompileTools.@setup_workflow?

sjdaines commented 1 year ago

Looks like this was just incorrect macro names introduce by the script(?) Should be setup_workload, compile_workload, not setup_workflow, precompile_workflow

timholy commented 1 year ago

Ah, I edited them by hand since the script wasn't designed to detect them without the @. Sorry about that, and thanks for figuring it out!