riogroup / SORA

MIT License
15 stars 12 forks source link

Fixed bugs in Star and prediction #48

Closed altairgomes closed 4 years ago

altairgomes commented 4 years ago
Bmorgado19 commented 4 years ago

Please, correct the docstring from Occultation(), now the time should be within 50 minutes range and not 10.

In Star() docstring please add if bjones default is True or False. Also, when bjones=True, please print a sentence similar to: Downloading star parameters from X/XXX/xxx, in the same manner as GDR2 and NOMAD.

altairgomes commented 4 years ago

I applied the suggested modifications, updated the changelog, and fixed a minor bug that happened when more than one star was found. This bug is because choice_star() require a "source" parameter stating which catalogue is being used for correct messaging. The parameter was not being passed by searchgaia() and getcolors().