Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update CI installation of h5py and petsc4py after release of setuptools 81.0#573

Merged
yguclu merged 9 commits into
develfrom
update_petsc_version
Feb 13, 2026
Merged

Update CI installation of h5py and petsc4py after release of setuptools 81.0#573
yguclu merged 9 commits into
develfrom
update_petsc_version

Conversation

@campospinto

@campospinto campospinto commented Feb 11, 2026

Copy link
Copy Markdown
Collaborator

On 06 Feb 2026 setuptools v81.0.0 was released with breaking changes, i.e. the support for the parameter --dry-run was removed from setup.py. As a consequence, our installed versions of h5py and petsc4py have stopped working. As a temporary fix for our CI workflows, we install the latest package versions from their respective repositories, where the issue has been fixed.

Detailed list of changes

  • Download h5py from GitHub (master branch) and build it from sources, instead of installing the latest release from PyPI.
  • Change the PETSc git clone command to download the 'release' branch instead of a broken version. Hence we install the latest package from the official GitLab repository. (When a new version number is available, we can point the git clone command to it.)
  • Trigger documentation workflow when PR is set to "Ready for review".

Updated the git clone command to use 'release' branch of petsc instead of specific version, because of an install problem with petsc4py
@campospinto campospinto marked this pull request as draft February 11, 2026 11:50
@campospinto campospinto marked this pull request as ready for review February 11, 2026 11:53
@codacy-production

codacy-production Bot commented Feb 11, 2026

Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 63223651
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (6322365) Report Missing Report Missing Report Missing
Head commit (ae88862) 63602 37634 59.17%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#573) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@yguclu yguclu marked this pull request as draft February 13, 2026 09:25
@yguclu yguclu marked this pull request as ready for review February 13, 2026 09:25
@yguclu yguclu changed the title Change PETSc clone command to use 'release' branch Update installation of h5py and pets4py after release of setuptools 81.0 Feb 13, 2026
@yguclu yguclu changed the title Update installation of h5py and pets4py after release of setuptools 81.0 Update installation of h5py and petsc4py after release of setuptools 81.0 Feb 13, 2026
@yguclu yguclu changed the title Update installation of h5py and petsc4py after release of setuptools 81.0 Update CI installation of h5py and petsc4py after release of setuptools 81.0 Feb 13, 2026

@yguclu yguclu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yguclu yguclu enabled auto-merge (squash) February 13, 2026 10:19
@yguclu yguclu marked this pull request as draft February 13, 2026 10:19
auto-merge was automatically disabled February 13, 2026 10:19

Pull request was converted to draft

@yguclu yguclu marked this pull request as ready for review February 13, 2026 10:19
@yguclu yguclu merged commit 812cf14 into devel Feb 13, 2026
9 checks passed
@yguclu yguclu deleted the update_petsc_version branch February 13, 2026 11:55
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