Julia 1.10 floor and dependency removal (breaking: Julia floor) - #137
Open
alecloudenback wants to merge 6 commits into
Open
alecloudenback wants to merge 6 commits into
alecloudenback wants to merge 6 commits into
Conversation
1.10 is the current LTS. The CI matrix now tests 1.10 and the latest release. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
Pkg was a hard runtime dependency used only for the artifact string macro, which the Artifacts stdlib provides directly. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
All 30 sites were in the parametric law catalogue. The laws whose parameters include a field named m now call their model argument model so the field does not shadow it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
The CSV loader now lives in ext/MortalityTablesCSVExt.jl and is activated by `using CSV`. The __init__ hook and the Requires dependency are gone. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
Parsed tables are cached by path in a Dict guarded by a ReentrantLock. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
Base.parse tolerates the surrounding whitespace found in the SOA XML and CSV files, and the full SOA test set passes unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Raises the minimum Julia to 1.10 (current LTS) and removes five dependencies. Six commits, one per item.
Changes
julia = "1.10"in Project.toml; CI matrix is1.10and1.Pkgreplaced by theArtifactsstdlib (it was a hard runtime dependency used only forartifact"...").UnPackreplaced by native(; a, b) = mdestructuring at all 30 sites. Laws with a field namedmnow call their model argumentmodelso the field does not shadow it.Requiresreplaced by a package extension: the CSV loader lives inext/MortalityTablesCSVExt.jland is activated byusing CSV.Memoizereplaced by an explicitDictcache guarded by aReentrantLockinreadXTbML.Parsersdropped in favor ofBase.parse, which tolerates the surrounding whitespace in the SOA files.Testing
readXTbMLreturns the identical object for repeated reads.Parsersremoval.Stacked on #136 (base branch
refactor/01-hygiene). Merge the stack in order; once the base PR is merged and its branch deleted, GitHub retargets this PR tomaster.🤖 Generated with Claude Code
https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT