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.

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

Phoenix SeqJSON timetags missing decimal point #1566

Closed
cartermak opened this issue Dec 2, 2024 · 0 comments · Fixed by #1567
Closed

Phoenix SeqJSON timetags missing decimal point #1566

cartermak opened this issue Dec 2, 2024 · 0 comments · Fixed by #1567
Labels
bug Something isn't working clipper Requests from the Europa Clipper project sequencing Anything related to the sequencing domain

Comments

@cartermak
Copy link
Member

Checked for duplicates

No - I haven't checked

Is this a regression?

Yes - This worked in a previous version

Version

3.0.1

Describe the bug

Generated SeqJSON command-relative and epoch-relative time tags are missing the decimal point when including milliseconds.

      {
        "tag": "00:06:40333", // Should be 00:06:40.333
        "type": "EPOCH_RELATIVE"
      }

I was able to work around this by adding a correction in the adaptation.

Reproduction

Using the default SeqN to SeqJSON converter in Phoenix, author a command-relative (R) or epoch-relative (E) time tagged command which specifies non-zero milliseconds.

Logs

No response

System Info

Aerie-UI 3.0.1

Severity

Minor

@cartermak cartermak added bug Something isn't working clipper Requests from the Europa Clipper project labels Dec 2, 2024
@cartermak cartermak added the sequencing Anything related to the sequencing domain label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working clipper Requests from the Europa Clipper project sequencing Anything related to the sequencing domain
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant