Skip to content

88 jd to hjd converter plugin is consistently different from other tools by a small amount#557

Merged
dbenn merged 4 commits intomasterfrom
88-jd-to-hjd-converter-plugin-is-consistently-different-from-other-tools-by-a-small-amount
Feb 23, 2026
Merged

88 jd to hjd converter plugin is consistently different from other tools by a small amount#557
dbenn merged 4 commits intomasterfrom
88-jd-to-hjd-converter-plugin-is-consistently-different-from-other-tools-by-a-small-amount

Conversation

@dbenn
Copy link
Collaborator

@dbenn dbenn commented Feb 18, 2026

@mpyat2, I have introduced changes to fix this long-standing JD to HJD conversion problem.

@clkotnik, I wanted to include you as a reviewer here (even if only for your awareness) since we had interactions about this bug including a forum post you wrote, an embarrassingly long time ago. I got there in the end. I probably should have more strongly considered your suggestion at the time of using alternative Java code.

In the end, my robot overlords (Cursor) helped me. This use of GenAI and agentic AI in particular is a sweet spot I think. Please see issue #88 for commentary.

…ethod in which the result was being converted to radians, which made no sense in the first place because eccentricity is a dimensionless measure of deviation from a circular orbit. The result of removing this improved some results and made some worse. The means by which this bug was found and the change was made is interesting, and I'll say more.
…tude was not being used. Again, in retrospect, it's unclear why I was not using it! A conversation with Cursor helped me worked through this. I made the relevant code and UT changes.
@dbenn dbenn requested review from clkotnik and mpyat2 February 18, 2026 13:09
@dbenn dbenn self-assigned this Feb 18, 2026
@dbenn dbenn added the bug Something isn't working label Feb 18, 2026
@mpyat2
Copy link
Collaborator

mpyat2 commented Feb 19, 2026

Hi @dbenn , please give me a couple of days. I’ll look at it over the weekend.

@dbenn
Copy link
Collaborator Author

dbenn commented Feb 19, 2026

Hi @dbenn , please give me a couple of days. I’ll look at it over the weekend.

Thanks @mpyat2 :)

Copy link
Collaborator

@mpyat2 mpyat2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @dbenn ,

Now the accuracy is quite good.

I took my observations of GSC_02743-01310 with time values converted from JD to HJD using AstroImageJ (see attachment: the first column is JD, the second is HJD converted by AstroImageJ)

Then I loaded the file to VStar (via Flexible Text plug-in) and converted the dates with the fixed plug-in. The absolute difference between VStar and AstroImageJ is less than 0.5s.
I think it's almost perfect.

Actually, the result could be even better because I rounded JD up to 5 digits after the point. In the original AstroImage file JD values have more digits (yet they are meaningless; they appear because the software I use -- SharpCap -- stores DATE-OBS with farctions of seconds).

GSC_02743-01310.tsv

@dbenn
Copy link
Collaborator Author

dbenn commented Feb 23, 2026

Thanks @mpyat2! Although more testing could be done, it seems we're in agreement that this branch can be merged.

@dbenn dbenn merged commit c97a026 into master Feb 23, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JD to HJD converter plugin is consistently different from other tools by a small amount

2 participants