IAU-ADES / ADES-Master

ADES implementation based on a master XML file
26 stars 7 forks source link

Correct time resolution for radar, and a few other items. #17

Closed stevechesley closed 1 year ago

stevechesley commented 1 year ago

Four items here:

The new transformation of times from decimal date to sexagesimal time in sexDateToISO() is fairly well tested. I wrote a script to check all possible fractional day values and found no issues with times like 03:60:60, etc...

@federicaspoto It looks like the changes you are making to mpc80coltoxml on your fix_python branch are compatible with this pull request.

federicaspoto commented 1 year ago

@stevechesley looks good to me! Feel free to merge the branch. I'll pull everything in our local copy.