Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bug: api.get_lightcurve should take in target name as well as targetid. #22

Open
emirkmo opened this issue Oct 10, 2022 · 0 comments
Open

Bug: api.get_lightcurve should take in target name as well as targetid. #22

emirkmo opened this issue Oct 10, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@emirkmo
Copy link
Member

emirkmo commented Oct 10, 2022 •
edited
Loading

tendrils/tendrils/api/lightcurves.py

Line 27 in da578e1

params = {'target': target}

Fix:
Modify api.get_lightcurve to take in target name as well as target id by querying for target id if target is not castable to int.
@emirkmo emirkmo added the bug Something isn't working label Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@emirkmo