Skip to content

Raise the julia compat lower bound of ControlSystemsBase to 1.10 - #1073

Merged
baggepinnen merged 1 commit into
masterfrom
controlsystemsbase-julia-compat
Oct 1, 2026
Merged

baggepinnen merged 1 commit into
masterfrom
controlsystemsbase-julia-compat

Conversation

@baggepinnen

Copy link
Copy Markdown
Member

Summary

The registration of ControlSystemsBase 1.22.1 (JuliaRegistries/General#170184) was not merged automatically, because the package could not be installed on julia 1.6.7, the lowest version allowed by its compat bound:

ERROR: Unsatisfiable requirements detected for package MatrixPencils [48965c70]:
 ├─restricted by julia compatibility requirements to versions: 1.0.0-1.7.13 or uninstalled
 └─restricted by compatibility requirements with ControlSystemsBase [aaaaaaaa] to versions: 1.8.3-1.9.1 — no versions left

ControlSystemsBase requires MatrixPencils ≥ 1.8.3, and all those versions require julia ≥ 1.8, so the declared bound julia = "1.6" cannot be satisfied. This PR raises the bound to the current LTS, julia = "1.10". The version remains 1.22.1, since 1.22.1 is not yet registered; Registrator will be retriggered on the merge commit.

Checked locally on julia 1.10.12: Pkg.develop of lib/ControlSystemsBase resolves against the current General registry and import ControlSystemsBase succeeds.

The top-level ControlSystems package also declares julia = "1.6". Since it depends on ControlSystemsBase, it has the same problem at its next registration. It is not changed here.


Host: demeter2, Claude Code session session_01Seo8ETEV7wrEk54KBJb7sy

🤖 Generated with Claude Code

https://claude.ai/code/session_01Seo8ETEV7wrEk54KBJb7sy

ControlSystemsBase requires MatrixPencils 1.8.3 or later, which requires
julia 1.8 or later, so the package could not be installed on julia 1.6.
The registration of ControlSystemsBase 1.22.1 was not merged
automatically for this reason.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Seo8ETEV7wrEk54KBJb7sy
@JuliaControlBot

Copy link
Copy Markdown

This is an automated message.
Plots were compared to references. 4/11 images have changed, see differences below.
After pulling this PR, please update the reference images by creating a PR to ControlExamplePlots.jl here.

Difference Reference Image New Image
✔️ 0.0 Reference New
✔️ 0.0 Reference New
✔️ 0.0 Reference New
✔️ 0.0 Reference New

@codecov

codecov Bot commented Oct 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.64%. Comparing base (2e023d9) to head (02582c3).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1073   +/-   ##
=======================================
  Coverage   91.64%   91.64%           
=======================================
  Files          42       42           
  Lines        5795     5795           
=======================================
  Hits         5311     5311           
  Misses        484      484           

☔ 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.

@baggepinnen
baggepinnen merged commit cd5fc44 into master Oct 1, 2026
6 checks passed
@baggepinnen
baggepinnen deleted the controlsystemsbase-julia-compat branch October 1, 2026 14:50
baggepinnen added a commit that referenced this pull request Oct 1, 2026
The julia compat lower bound was raised to 1.10 in #1073, which the
General registry does not accept in a patch release.


Claude-Session: https://claude.ai/code/session_01Seo8ETEV7wrEk54KBJb7sy

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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