Skip to content

Conversation

@zemogle
Copy link
Contributor

@zemogle zemogle commented Apr 25, 2015

This request only has basic image meta data and photometry querying. You can search by name and region. This is for the LCOGT archive which is hosted by IPAC and updated daily.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.32%) to 66.16% when pulling 7a11c5e on zemogle:lcogt into 7fdb127 on astropy:master.

duplicating code

admittedly, this results in slightly less coherent docstrings, but it
helps reduce code duplication, which is perhaps more important.  Tests
still pass!
@keflavich
Copy link
Contributor

@zemogle I've made a PR onto this. I'm happy to merge once you merge that.

Thanks for refactoring. Useful to know those tips.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.31%) to 66.15% when pulling 97e5c90 on zemogle:lcogt into 7fdb127 on astropy:master.

@keflavich
Copy link
Contributor

Merging, but we need to add docs: lcogt/lcogt.rst and an entry in index.rst!

keflavich added a commit that referenced this pull request Apr 25, 2015
Adding LCOGT archive support
@keflavich keflavich merged commit ebf8de5 into astropy:master Apr 25, 2015
@zemogle
Copy link
Contributor Author

zemogle commented Apr 25, 2015

Thanks. Its always the docs that get forgotten. I'll get on to that this weekend.

@bsipocz bsipocz mentioned this pull request Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants