fraserw / trippy

Python based Trailed Source Photometry
GNU General Public License v3.0
20 stars 13 forks source link

Push version 1.2 to Pypi? #39

Open Mikea1985 opened 4 years ago

Mikea1985 commented 4 years ago

After a collaborator tried to install Trippy (from Pypi, not from GitHub) and was having issues, I looked into this. The following reproduces the procedure and error that my collaborator got:

conda create -n trippyconda37 python=3.7 numpy stsci matplotlib astropy sextractor emcee numba pytest
conda activate trippyconda37
pip install trippy
python -c "import numpy, scipy, matplotlib, astropy, emcee, numba"
python -c "import trippy; from trippy import bgFinder, psfStarChooser, scamp, MCMCfit"
cd
git clone https://github.com/fraserw/trippy.git
cd trippy/trippy/tests
python tests.py

Which results in the following error: fail.txt

The same happened if I tried with Python 3.6 rather than 3.7. I believe that what the error message means is that a numpy float is used as an integer index, which is of course a no-no now that it's apparently no longer obvious how to turn a float into an integer. When I remove the pip installed trippy and instead soft-link to the version in the git directory, it all works. Turns out pip is still installing version 1.1.1 whereas here on GitHub we are at 1.2. I assume that the problem with 1.1.1 is only relevant for some versions of Numpy or Python, but rather than looking into that, perhaps its easier to just push the version that does work to Pypi?

fraserw commented 4 years ago

Thanks for the heads up! I agree uploading a new version to pip is a good idea. Will do so in the next day or so, and let you know when its done.

On Jun 1, 2020, at 12:28 PM, Mike Alexandersen notifications@github.com wrote:

After a collaborator tried to install Trippy (from Pypi, not from GitHub) and was having issues, I looked into this. The following reproduces the procedure and error that my collaborator got:

conda create -n trippyconda37 python=3.7 numpy stsci matplotlib astropy sextractor emcee numba pytest conda activate trippyconda37 pip install trippy python -c "import numpy, scipy, matplotlib, astropy, emcee, numba" python -c "import trippy; from trippy import bgFinder, psfStarChooser, scamp, MCMCfit" cd git clone https://github.com/fraserw/trippy.git cd trippy/trippy/tests python tests.py Which results in the following error: fail.txt https://github.com/fraserw/trippy/files/4712993/fail.txt The same happened if I tried with Python 3.6 rather than 3.7. I believe that what the error message means is that a numpy float is used as an integer index, which is of course a no-no now that it's apparently no longer obvious how to turn a float into an integer. When I remove the pip installed trippy and instead soft-link to the version in the git directory, it all works. Turns out pip is still installing version 1.1.1 whereas here on GitHub we are at 1.2. I assume that the problem with 1.1.1 is only relevant for some versions of Numpy or Python, but rather than looking into that, perhaps its easier to just push the version that does work to Pypi?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/fraserw/trippy/issues/39, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRJZPR5NRTZCFWYI4R2MKDRUP6M5ANCNFSM4NQCQOMA.

Mikea1985 commented 4 years ago

I think I remember there being an outstanding pull request from me. In not sure whether that should be integrated before pushing to pip.

On Mon, Jun 1, 2020, 15:36 Wesley Fraser notifications@github.com wrote:

Thanks for the heads up! I agree uploading a new version to pip is a good idea. Will do so in the next day or so, and let you know when its done.

On Jun 1, 2020, at 12:28 PM, Mike Alexandersen notifications@github.com wrote:

After a collaborator tried to install Trippy (from Pypi, not from GitHub) and was having issues, I looked into this. The following reproduces the procedure and error that my collaborator got:

conda create -n trippyconda37 python=3.7 numpy stsci matplotlib astropy sextractor emcee numba pytest conda activate trippyconda37 pip install trippy python -c "import numpy, scipy, matplotlib, astropy, emcee, numba" python -c "import trippy; from trippy import bgFinder, psfStarChooser, scamp, MCMCfit" cd git clone https://github.com/fraserw/trippy.git cd trippy/trippy/tests python tests.py Which results in the following error: fail.txt https://github.com/fraserw/trippy/files/4712993/fail.txt The same happened if I tried with Python 3.6 rather than 3.7. I believe that what the error message means is that a numpy float is used as an integer index, which is of course a no-no now that it's apparently no longer obvious how to turn a float into an integer. When I remove the pip installed trippy and instead soft-link to the version in the git directory, it all works. Turns out pip is still installing version 1.1.1 whereas here on GitHub we are at 1.2. I assume that the problem with 1.1.1 is only relevant for some versions of Numpy or Python, but rather than looking into that, perhaps its easier to just push the version that does work to Pypi?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/fraserw/trippy/issues/39>, or unsubscribe < https://github.com/notifications/unsubscribe-auth/ABRJZPR5NRTZCFWYI4R2MKDRUP6M5ANCNFSM4NQCQOMA .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fraserw/trippy/issues/39#issuecomment-637060935, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADRV7CDS64EH4SXTVGN4VL3RUP7KLANCNFSM4NQCQOMA .

fraserw commented 4 years ago

Got a number? I thought I had sorted them all...

On Sat., Jun. 6, 2020, 9:20 a.m. Mike Alexandersen notifications@github.com wrote:

I think I remember there being an outstanding pull request from me. In not sure whether that should be integrated before pushing to pip.

On Mon, Jun 1, 2020, 15:36 Wesley Fraser notifications@github.com wrote:

Thanks for the heads up! I agree uploading a new version to pip is a good idea. Will do so in the next day or so, and let you know when its done.

On Jun 1, 2020, at 12:28 PM, Mike Alexandersen < notifications@github.com> wrote:

After a collaborator tried to install Trippy (from Pypi, not from GitHub) and was having issues, I looked into this. The following reproduces the procedure and error that my collaborator got:

conda create -n trippyconda37 python=3.7 numpy stsci matplotlib astropy sextractor emcee numba pytest conda activate trippyconda37 pip install trippy python -c "import numpy, scipy, matplotlib, astropy, emcee, numba" python -c "import trippy; from trippy import bgFinder, psfStarChooser, scamp, MCMCfit" cd git clone https://github.com/fraserw/trippy.git cd trippy/trippy/tests python tests.py Which results in the following error: fail.txt https://github.com/fraserw/trippy/files/4712993/fail.txt The same happened if I tried with Python 3.6 rather than 3.7. I believe that what the error message means is that a numpy float is used as an integer index, which is of course a no-no now that it's apparently no longer obvious how to turn a float into an integer. When I remove the pip installed trippy and instead soft-link to the version in the git directory, it all works. Turns out pip is still installing version 1.1.1 whereas here on GitHub we are at 1.2. I assume that the problem with 1.1.1 is only relevant for some versions of Numpy or Python, but rather than looking into that, perhaps its easier to just push the version that does work to Pypi?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/fraserw/trippy/issues/39>, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ABRJZPR5NRTZCFWYI4R2MKDRUP6M5ANCNFSM4NQCQOMA

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fraserw/trippy/issues/39#issuecomment-637060935, or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADRV7CDS64EH4SXTVGN4VL3RUP7KLANCNFSM4NQCQOMA

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/fraserw/trippy/issues/39#issuecomment-640084548, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRJZPU5XHDDRGAGRBFV3PDRVJUE7ANCNFSM4NQCQOMA .

Mikea1985 commented 4 years ago

Pull request #36 seems to still be open. It exposes the ability to MCMC fit different parameters independently more easily, and explains it in the tutorial.

fraserw commented 4 years ago

Yep that’s right, because I haven’t had time to look at TRIPPy for quite a while...

On Jun 19, 2020, at 9:27 AM, Mike Alexandersen notifications@github.com wrote:

Pull request #36 https://github.com/fraserw/trippy/pull/36 seems to still be open. It exposes the ability to MCMC fit different parameters independently more easily, and explains it in the tutorial.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/fraserw/trippy/issues/39#issuecomment-646729193, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRJZPRCXONG44F7EQQNYZTRXOGX7ANCNFSM4NQCQOMA.