Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hcipy: init at 0.6.0 #341906

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

hcipy: init at 0.6.0 #341906

wants to merge 1 commit into from

Conversation

BarrOff
Copy link
Contributor

@BarrOff BarrOff commented Sep 14, 2024

This PR adds the HCIPy python library.
HCIPy is an open-source, modular Python package for high-contrast imaging on current and future telescopes.
More information available on the projects homepage.

I have successfully built the package and tested it's functionality.

This is the first time I add a Python package.
Help is really appreciated if I missed something.

Things done

  • Built on platform(s)
    • x86_64-linux

@BarrOff BarrOff requested a review from natsukium as a code owner September 14, 2024 19:39
@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 labels Sep 15, 2024
@BarrOff BarrOff force-pushed the add-hcipy branch 2 times, most recently from a5e3eab to 09457e0 Compare September 15, 2024 15:55
Copy link
Member

@FliegendeWurst FliegendeWurst left a comment

Choose a reason for hiding this comment

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

The commit message should start with python3Packages.

pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/hcipy/default.nix Outdated Show resolved Hide resolved
@BarrOff BarrOff force-pushed the add-hcipy branch 2 times, most recently from abf582f to a5aaa85 Compare November 3, 2024 14:50
@BarrOff
Copy link
Contributor Author

BarrOff commented Nov 3, 2024

The commit message should start with python3Packages.

I have adjusted the commit message and applied all requested changes.

@FliegendeWurst FliegendeWurst added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Nov 5, 2024
@BarrOff BarrOff force-pushed the add-hcipy branch 2 times, most recently from def1d53 to 935253e Compare November 9, 2024 23:57
@wegank wegank removed the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Nov 10, 2024
@FliegendeWurst
Copy link
Member

@ofborg build python312Packages.hcipy

Copy link
Member

@FliegendeWurst FliegendeWurst left a comment

Choose a reason for hiding this comment

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

Tests fail on aarch64-linux

tests/test_aperture.py ................................................. [ 13%]
.............s.s..............                                           [ 21%]
tests/test_atmosphere.py .s.s.s.s......                                  [ 25%]
tests/test_config.py .                                                   [ 26%]
tests/test_coronagraphy.py ........Fatal Python error: Aborted

Please mark as broken or fix

@BarrOff
Copy link
Contributor Author

BarrOff commented Nov 10, 2024

Tests fail on aarch64-linux

tests/test_aperture.py ................................................. [ 13%]
.............s.s..............                                           [ 21%]
tests/test_atmosphere.py .s.s.s.s......                                  [ 25%]
tests/test_config.py .                                                   [ 26%]
tests/test_coronagraphy.py ........Fatal Python error: Aborted

Please mark as broken or fix

I'm neither a developer of the project nor do I have arm64 hardware to check and debug the problem, so I can't fix it.
Therefore I marked the package as broken on arm64 for now and created an issue on the upstream project.

@FliegendeWurst FliegendeWurst added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Nov 10, 2024
setuptools-scm,

# dependencies
numpy,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
numpy,
numpy_1,

Package is not compatible with 2.x, which will be the new default very soon.

@wegank wegank removed the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: python 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants