Skip to content

Pyjapc pin and warning#113

Merged
fsoubelet merged 4 commits intomasterfrom
pyjapc_warn
Mar 12, 2026
Merged

Pyjapc pin and warning#113
fsoubelet merged 4 commits intomasterfrom
pyjapc_warn

Conversation

@fsoubelet
Copy link
Member

The pyjapc package, which is part of the [cern] optional dependencies, has reached its end-of-life.

I have pinned the dependency on python_version < 3.14 and added a warning straight away in brst_logger.py to provide information on this. Realistically, the BRST logger script has not been used in the past 5 years so I don't expect much disruption.

This would be a patch release.

Note: I have pinned < 3.14 for now but I'm waiting on Phil Elson or others to clarify which Python version exactly is the last supported one, officially. I might change this pin again.

With regards to the Acc-Py distributions, considering the last one was 3.11-based (supported) and the new one is 3.14-based (unsupported) this still works for now.

@fsoubelet fsoubelet requested a review from jgray-19 March 12, 2026 10:55
@fsoubelet fsoubelet self-assigned this Mar 12, 2026
@fsoubelet fsoubelet added Type: Bug Something isn't working as it should. Type: Release Issue preparing for a release. Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Priority: Medium Work on this. Type: Maintenance Improvements in the code, that are not necessarily visible in functionality. Status: Review Needed Work currently stopped, untils someone else reviews it. labels Mar 12, 2026
@github-actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  pylhc
  __init__.py
  bsrt_logger.py 15-30
Project Total  

This report was generated by python-coverage-comment-action

@fsoubelet
Copy link
Member Author

After confirming with Ivan Sinkarenko I have pinned python_version < '3.12'

@fsoubelet
Copy link
Member Author

Merging as the low coverage is of no impact on this package: almost everything here is CERN-network specific and can't run on this CI.

@fsoubelet fsoubelet merged commit 03fd4dd into master Mar 12, 2026
22 of 23 checks passed
@fsoubelet fsoubelet deleted the pyjapc_warn branch March 12, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Priority: Medium Work on this. Status: Review Needed Work currently stopped, untils someone else reviews it. Type: Bug Something isn't working as it should. Type: Maintenance Improvements in the code, that are not necessarily visible in functionality. Type: Release Issue preparing for a release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants