search

quatrope / astroalign

A tool to align astronomical images based on asterism matching
MIT License
143 stars 44 forks source link

test without sep installed #4

Closed martinberoiz closed 7 years ago

martinberoiz commented 8 years ago

Make travis env to test without sep installed

leliel12 commented 8 years ago

que es env?

leliel12 commented 8 years ago

por ahi no entendi bien pero en tu tox.ini vi eso pip install sep por que sep no esta en la seccion deps de [testenv]

martinberoiz commented 8 years ago

env=environment si, con el sep no se bien que hacer. Es una "dependencia" que no es necesaria, si esta la uso, y si no esta uso otro método casero. Pero pasa que sí la uso en los tests :/ Entonces quiero testear un environment donde no tenga sep instalado para estar seguro que defaultea al findsources casero y q anda bien

leliel12 commented 8 years ago

ponela en "deps" que solo instala para el test env. Fijate que en corral si encuentra ipython lo usa... sino se va al interprete clásico

On Mon, 25 Jan 2016 at 11:06 Martin Beroiz notifications@github.com wrote:

env=environment si, con el sep no se bien que hacer. Es una "dependencia" que no es necesaria, si esta la uso, y si no esta uso otro método casero. Pero pasa que sí la uso en los tests :/ Entonces quiero testear un environment donde no tenga sep instalado para estar seguro que defaultea al findsources casero y q anda bien

— Reply to this email directly or view it on GitHub https://github.com/toros-astro/astroalign/issues/4#issuecomment-174518099 .

Juan BC (from phone)

martinberoiz commented 8 years ago

ahi me fijo como hiciste, gracias

leliel12 commented 8 years ago

ponelo en deps de tox.ini

On Mon, 25 Jan 2016 at 13:24 Martin Beroiz notifications@github.com wrote:

ahi me fijo como hiciste, gracias

— Reply to this email directly or view it on GitHub https://github.com/toros-astro/astroalign/issues/4#issuecomment-174564524 .

Juan BC (from phone)