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

Add dev versions of Brightway packages to conda-forge Channel #19

Closed
michaelweinold opened this issue Aug 17, 2023 · 4 comments
Closed
Assignees
Labels
enhancement New feature or request infrastructure

Comments

@michaelweinold
Copy link
Collaborator

michaelweinold commented Aug 17, 2023

As per

we can only use the conda-forge and emscripten-forge channels to build the xeus-python kernel at the moment:

Only no-arch packages from conda-forge and emscripten-forge can be installed.

While I was unable to find specific guidance in the conda-forge documentation, it seems (cf. the pandas package) that packages on the conda-forge channel can have development versions also.

Official guidance by the conda-forge team is provided in CFEP05 (according to answers to my question in the conda-forge Element channel).

We should therefore add the dev versions, which are currently only on the cmutel channel to the conda-forge channel.

@michaelweinold
Copy link
Collaborator Author

michaelweinold commented Aug 17, 2023

@tngTUDOR
Copy link

AS from https://github.com/conda-forge/bw2data-feedstock#updating-bw2data-feedstock for example, we might need to fork+branch to automatically create new packages

@michaelweinold
Copy link
Collaborator Author

to automatically create new packages

At this point, I would recommend we add only those latest dev versions that have been confirmed to work with my WASM setup.

We can think about an automated workflow at a later point. This issue really is just to work around what is currently an annoying limitation in the xeus-python-kernel (which might well have a reasonable explanation).

@michaelweinold
Copy link
Collaborator Author

This should do it - many thanks to @m-rossi for the helpful comments and lightning-fast approval of the first PR (and thank you to @cmutel for the other two)!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants