Skip to content

Firefly-1973: add ecliptic coordinate system to projection for HiPS#1929

Merged
robyww merged 1 commit intodevfrom
FIREFLY-1973-hips-ecl
Apr 3, 2026
Merged

Firefly-1973: add ecliptic coordinate system to projection for HiPS#1929
robyww merged 1 commit intodevfrom
FIREFLY-1973-hips-ecl

Conversation

@robyww
Copy link
Copy Markdown
Contributor

@robyww robyww commented Apr 2, 2026

Firefly-1973: add ecliptic coordinate system to projection for HiPS

  • now implements and option to show HiPS in ecliptic

Testing

@robyww robyww added this to the 2026.1 milestone Apr 2, 2026
@robyww robyww self-assigned this Apr 2, 2026
@robyww robyww requested review from kpuriIpac and lrebull April 2, 2026 20:44
@lrebull
Copy link
Copy Markdown
Contributor

lrebull commented Apr 2, 2026

It's really cool but i managed to thoroughly confuse myself and i'm not sure now if i found a bug or not. i do think something is inconsistent. see video.

video1225794930.mp4

Copy link
Copy Markdown
Contributor

@kpuriIpac kpuriIpac left a comment

Choose a reason for hiding this comment

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

Looks good.

@kpuriIpac
Copy link
Copy Markdown
Contributor

It's really cool but i managed to thoroughly confuse myself and i'm not sure now if i found a bug or not. i do think something is inconsistent. see video.

video1225794930.mp4

From the video (towards the end when you ran into the issue) I can see something is inconsistent, but I wasn't able to reproduce this bug.

@robyww robyww merged commit e42b664 into dev Apr 3, 2026
@robyww robyww deleted the FIREFLY-1973-hips-ecl branch April 3, 2026 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants