EranOfek / AstroPack

Astronomy & Astrophysics Software Pacakge
Other
16 stars 4 forks source link

`celestial.Targets.isVisible` for negative HA bug? #447

Closed EastEriq closed 2 months ago

EastEriq commented 2 months ago

In obsByPriority2 I found this comment

    % disable bug in Targets.isVisible which can't deal with negative HA,
    % remove line ones that bug is fixed
    T.VisibilityArgs.HALimit=400;

what is the current status of it? Was the bug solved?

EranOfek commented 2 months ago

I am not aware of any such bug. Just gave a look into the code and couldn't find evidence for this problem. Minor updates to help (new version). [dev1 473c95ac]

On Wed, May 22, 2024 at 3:10 PM EastEriq @.***> wrote:

In obsByPriority2 I found this comment

% disable bug in Targets.isVisible which can't deal with negative HA,
% remove line ones that bug is fixed

what is the current status of it? Was the bug solved?

— Reply to this email directly, view it on GitHub https://github.com/EranOfek/AstroPack/issues/447, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJUQ4JYUZ5VYDRFIQQIRBTZDSDMRAVCNFSM6AAAAABIDQD6O2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYTAMZXGIZTSMA . You are receiving this because you are subscribed to this thread.Message ID: @.***>