Skip to content

Standardize naming and use master checkouts for workspace dependencies - #12

Merged
mlakolar merged 3 commits into
masterfrom
codex/naming-master-dependencies
Oct 25, 2026
Merged

mlakolar merged 3 commits into
masterfrom
codex/naming-master-dependencies

Conversation

@mlakolar

@mlakolar mlakolar commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Use snake_case for descriptive estimation, bootstrap, and inverse-Hessian APIs and their source files. Preserve deprecated public function bindings and old keyword/property spellings. Update experiments and tests. Current master already removed its CoordinateDescent dependency, so this change adds no new HighDimensionalModels dependency.

Develop workspace dependencies recursively from sibling master checkouts, including transitive dependencies and test extras on Julia 1.10. Add the master bootstrap/test runner, naming guard, and matching CI/development instructions. Package versions and UUIDs are unchanged; historical benchmark environments/results remain frozen.

Prerequisite migration PRs: CovarianceSelection.jl #33.

Coordinated merge order: ProximalBase.jl #111 and KernelSmoothing.jl #78; then ProximalSolvers.jl #93 and FunctionalDataAnalysis.jl #32; then HighDimensionalModels.jl #95; then CovarianceSelection.jl #33; finally DiffPrecTest.jl #12 and FunctionalGraphicalModels.jl #11.

Validation: Linux LTS/current, macOS, Windows, and both Codecov checks pass after CovarianceSelection #33 merged. Gaussian covariance-difference variance is checked against the independent Kronecker/commutation identity, including permuted and repeated indices and an empty selection. This covers the previously untested renamed helper without relaxing coverage thresholds.

@codecov

codecov Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.63946% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.62%. Comparing base (d7eef6d) to head (0ad3f58).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/DiffPrecTest.jl 96.72% 2 Missing ⚠️
src/reduced_diff_estim.jl 96.55% 1 Missing ⚠️
src/util.jl 97.43% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   98.62%   99.62%   +1.00%     
==========================================
  Files          13       14       +1     
  Lines        1306     1328      +22     
==========================================
+ Hits         1288     1323      +35     
+ Misses         18        5      -13     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mlakolar
mlakolar marked this pull request as ready for review October 25, 2026 14:06
@mlakolar
mlakolar merged commit 6ba332c into master Oct 25, 2026
6 checks passed
@mlakolar
mlakolar deleted the codex/naming-master-dependencies branch October 25, 2026 16:01
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