-
Notifications
You must be signed in to change notification settings - Fork 4
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
PyPI Package? #5
Comments
Dear @Zeitsperre, Thanks for your contribution, your PR is now merged. I had planned to make SBCK installable by pip and conda once the XSBCK ( |
5 tasks
Zeitsperre
added a commit
to Ouranosinc/xclim
that referenced
this issue
Mar 31, 2023
### What kind of change does this PR introduce? * Grammar and spelling mistakes within the notebooks have been ~corrected~ reduced. * `SBCK` has been pulled from `environment.yml` for the time being until a proper package is created. * The Contributing guide now lists three separate subsections to help users understand the gains from optional dependencies ### Does this PR introduce a breaking change? Yes, but no[^1]. Changes to dependencies have been handled in such a way that default settings are unaffected. Development and CI-based behaviour is affected: `clisops` and `flox` are now in the `environment.yml`. ### Other information: yrobink/SBCK-python#5 [^1]: https://i.kym-cdn.com/entries/icons/original/000/028/596/dsmGaKWMeHXe9QuJtq_ys30PNfTGnMsRuHuo_MUzGCg.jpg
Dear @Zeitsperre , SBCK is now available in pypi: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @yrobink,
I'm wondering if you would be interested in having SBCK-python as a
pip
-installable Python package on PyPI. My PR (#4) fixes most of the issues that would block this package from being contributed, and the remaining work involved is miniscule (creating a user, generating a source package, uploading to PyPI withtwine
).If this is of interest to you, I don't mind contributing some time towards this. This would be a welcome addition for
xclim
.The text was updated successfully, but these errors were encountered: