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

Test conda package build with NEST master branch integrated in CI #2918

Closed
wants to merge 53 commits into from

Conversation

steffengraber
Copy link
Contributor

The building of the package is kept as close as possible to the conda-forge package building. The tooling also corresponds to that of conda-forge.
The build runs under Ubuntu 22.04 with python3.10. But this can be changed or extended. Tests with MacOSX are also generally possible.

The build package is saved as an artifact under nest_conda_Linux_x86_64 and can be tested afterward.
To do this, download the zip file, unzip it and thus install NEST with the appropriate environment:

mamba create -n MY-ENV -c "file://${PWD}/nest_conda_Linux_x86_64" nest-simulator

This also works with conda instead of mamba.

@steffengraber steffengraber added T: Enhancement New functionality, model or documentation S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels Aug 31, 2023
@steffengraber
Copy link
Contributor Author

API calls to start and test the CI workflow for NEST are now integrated.
@terhorstd and @gtrensch Some secrets need to be set, and we should discuss the integration in the NEST CI workflow.

@heplesser heplesser self-requested a review November 16, 2023 12:46
@heplesser
Copy link
Contributor

@steffengraber This looks fine in principle, but the packaging test fails. Could you check that?

@heplesser heplesser requested a review from terhorstd March 1, 2024 15:15
@steffengraber
Copy link
Contributor Author

The whole project is being carefully reconsidered. There should be better ways to integrate CIs. Therefore this PR will be closed.

@terhorstd terhorstd removed the T: Maintenance Work to keep up the quality of the code and documentation. label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Enhancement New functionality, model or documentation
Projects
Development

Successfully merging this pull request may close these issues.

3 participants