search

panoptes / panoptes-pipeline

PANOPTES Data Processing Pipeline
MIT License
9 stars 8 forks source link

add tests for light curve simulator and combiner and minor updates #21

Closed kstoreyf closed 8 years ago

kstoreyf commented 8 years ago
wtgee commented 8 years ago

Try adding this to the .travis.yml for your branch in the install section:

pip -q install git+http://github.com/panoptes/POCS.git#egg=pocs --no-deps

kstoreyf commented 8 years ago

tried that, idk...

wtgee commented 8 years ago

Move it up above line 28, so it installs POCS before it tries the python setup.py install for PIAA

On Sat, Aug 6, 2016 at 11:19 AM, kstoreyf notifications@github.com wrote:

tried that, idk...

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/panoptes/PIAA/pull/21#issuecomment-237997985, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEUUJJQxJUpysswGA9yw44VRkDIidjdks5qc-E3gaJpZM4JeKSr .

~Wilfred Tyler Gee

kstoreyf commented 8 years ago

haha thanks for all these, sorry I'm away from comp now

wtgee commented 8 years ago

No worries. I should have had it all fixed up. It's nothing to do with your code but with the travis-ci setup. I already struggled through it all with POCS so I figured it would be easier for me to just get it going.

On Sat, Aug 6, 2016 at 12:24 PM, kstoreyf notifications@github.com wrote:

haha thanks for all these, sorry I'm away from comp now

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/panoptes/PIAA/pull/21#issuecomment-238001084, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEUUAVj2GtfMxIpTymoDysK5JJ9qMrIks5qc_BfgaJpZM4JeKSr .

~Wilfred Tyler Gee

wtgee commented 8 years ago

Ok, this is now failing on the actual tests rather than on misconfigured travis build. I'm handing it back to you @kstoreyf :)