You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'd like to release current master, or something close to it, as 0.3.5. Our current PyPI release, 0.3.4, is arguably broken: pip install slycot doesn't work on Linux systems (I suspect also not on OSX), but 0.3.5 should fix that.
@murrayrm, could you please test the build on OSX? Something like pip install git+https://github.com/python-control/Slycot in a clean virtual-env (no numpy, no scikit-build, up-to-date pip) should be sufficient.
AFAIK, conda builds are still OK. @repagh, @moorepants, please let me know if there is anything conda-related that must be done.
I'd use the conda-forge docs as the primary source for the conda-forge feedstock and the conda-build for anything you are doing in this repo. Conda-forge has some requirements that are specific to their build infrastructure.
I'd use the conda-forge docs as the primary source for the conda-forge feedstock and the conda-build for anything you are doing in this repo.
My understanding of why we're doing conda-builds at all in our CI is as early-warning of problems for conda-forge, so we might as well follow conda-forge practices.
Speaking of which, is there any way to use the conda-forge recipe in the Slycot, rather than using our own? It feels like we're duplicating effort i.t.o. of conda-build recipe maintenance otherwise.
The conda recipes in the Slycot repo predated me building it on conda forge. I'm not sure I've ever looked at them. If you want to build the conda forge recipes before a release, then you may have to set up a similar CI configuration. It is possible to have conda forge build prereleases from a sub-channel which is another way to trigger a conda forge build on their system and avoiding people installing a new release.
Activity
bnavigator commentedon Jun 21, 2019
Current master builds fine on the OpenSUSE build system now, thanks for the fixes!
https://build.opensuse.org/package/show/home:bnavigator/python-slycot
moorepants commentedon Jun 21, 2019
One notes is that we should change build.sh and bld.bat in the conda-forge feedstock to the recommended:
Then pip install-ability will be checked downstream. You can do something similar on the Travis builds here.
roryyorke commentedon Jun 22, 2019
@moorepants thanks, that's a good idea. From googling (e.g. conda-build docs and a conda-forge tutorial) it looks like one should add pip to the list of
hostrequirements in that case.Is there any somewhat-authoritative source for this recommendation? Would like to put it in the commit log.
roryyorke commentedon Jun 22, 2019
current conda-forge docs recommend
"{{ PYTHON }} -m pip install . --no-deps -vv"The "--ignore" makes sense, so I'll keep that; I'll add -vv.
moorepants commentedon Jun 22, 2019
I'd use the conda-forge docs as the primary source for the conda-forge feedstock and the conda-build for anything you are doing in this repo. Conda-forge has some requirements that are specific to their build infrastructure.
moorepants commentedon Jun 22, 2019
This is what I was looking for originally:
I found
python -m pip install --no-deps --ignore-installed .instead. The docs are too confusing! Go with the one you found.roryyorke commentedon Jun 23, 2019
My understanding of why we're doing conda-builds at all in our CI is as early-warning of problems for conda-forge, so we might as well follow conda-forge practices.
Speaking of which, is there any way to use the conda-forge recipe in the Slycot, rather than using our own? It feels like we're duplicating effort i.t.o. of conda-build recipe maintenance otherwise.
moorepants commentedon Jun 23, 2019
The conda recipes in the Slycot repo predated me building it on conda forge. I'm not sure I've ever looked at them. If you want to build the conda forge recipes before a release, then you may have to set up a similar CI configuration. It is possible to have conda forge build prereleases from a sub-channel which is another way to trigger a conda forge build on their system and avoiding people installing a new release.
murrayrm commentedon Jun 23, 2019
Report on OSX build:
Trying to build from a completely clean environment (Python 3.7, using conda) fails because of the lack of a FORTRAN compiler.
After installing a FORTRAN compiler (using
conda install gcc), installation was successful. Output attached below.roryyorke commentedon Jun 23, 2019
@murrayrm great, thank you. 0.3.5 is tagged - could you please publish to PyPI?
murrayrm commentedon Jun 23, 2019
Uploaded. SHA256 has is
cad98d5ea4f0a034cf398c39189f587620a0b03f1d4b71e77cd622a327f13adf