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

error on prediction #16

Closed
FlaviaLu opened this issue May 22, 2020 · 2 comments · Fixed by #21
Closed

error on prediction #16

FlaviaLu opened this issue May 22, 2020 · 2 comments · Fixed by #21
Assignees
@altairgomes
Labels
bug Something isn't working effort_low Probably easy to solve prediction
Milestone
V0.1.1

Comments

@FlaviaLu
Copy link
Collaborator

When I run the prediction for a short time interval (3h) for 2002KX14 occultation in 2018, I got a TypeError: 'Longitude' object with a scalar value does not support indexing. The code runs well when I delete the mag_lim=18 and put a bigger interval of time to search for occultation (some days).
@altairgomes altairgomes added bug Something isn't working effort_low Probably easy to solve prediction labels May 22, 2020
@altairgomes altairgomes added this to the V0.1.1 milestone May 22, 2020
@altairgomes altairgomes self-assigned this May 22, 2020
@altairgomes
Copy link
Collaborator

This happens when only one occultation is found. It is a bug very easy to solve.

@altairgomes altairgomes mentioned this issue May 27, 2020
Merged
@altairgomes altairgomes linked a pull request May 27, 2020 that will close this issue
Modifications using PEP8 and minor corrections. #21
Merged
@altairgomes
Copy link
Collaborator

This issue has been fixed and merged to the develop branch. It will be available in v0.1.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@altairgomes altairgomes

Labels
bug Something isn't working effort_low Probably easy to solve prediction
Projects
None yet
Milestone
V0.1.1
Development

Successfully merging a pull request may close this issue.

Modifications using PEP8 and minor corrections.
2 participants
@altairgomes @FlaviaLu