Skip to content

Euclid MER catalog CMD: Reverse y-axis and zoom in#104

Merged
bsipocz merged 2 commits intoCaltech-IPAC:mainfrom
troyraen:raen/patch/euclid-mer-catalog
May 15, 2025
Merged

Euclid MER catalog CMD: Reverse y-axis and zoom in#104
bsipocz merged 2 commits intoCaltech-IPAC:mainfrom
troyraen:raen/patch/euclid-mer-catalog

Conversation

@troyraen
Copy link
Copy Markdown
Contributor

In the Euclid MER catalog CMD figure, reverse the y-axis so that brighter stars are on top and zoom in to see detail a little better. The new figure looks like this (I can adjust x-and y-limits differently if wanted):

cmd-10k-stars-in-mer

@troyraen troyraen requested a review from jkrick May 15, 2025 19:34
@bsipocz bsipocz added content Content related issues/PRs. maintenance General maintenance of the content and/or infrastructure labels May 15, 2025
Copy link
Copy Markdown
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

Good eyes, thanks for spotting this!

Copy link
Copy Markdown
Contributor

@jkrick jkrick left a comment

Choose a reason for hiding this comment

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

Thanks for doing this,
I suggest xlim=(-2, 2), ylim=(24, 16)

Then, if you really want to have fun, if there are still too many points to see what is going on in the blob, switch it to some kind of heatmap, or such, but this is completely unnecessary.

@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented May 15, 2025

I don't want to bike-shed as it doesn't matter for this notebook, but is the gap for the very faint objects is real or is an artifact? Also, would the very red ones be an interesting group or they are just artefact again?

(That said, the cuts to zoom on the main locus in the notebook makes most sense)

@troyraen
Copy link
Copy Markdown
Contributor Author

Changed the limits and the new figure looks like this, and I'll leave it there for now.

cmd-10k-stars-in-mer

@bsipocz I really don't know. My naive guess is that maybe the really faint clump is white dwarfs (though not sure that they're quite in the right place for that) and the slightly brighter but red ones are pre-main sequence stars?

I'll merge once the 2 pending checks come back.

@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented May 15, 2025

The two pending checks will fail, so no need to wait for them. Also, apparently we have #105.

@troyraen
Copy link
Copy Markdown
Contributor Author

@bsipocz I don't have permission to merge while checks are pending nor when checks fail, so actually I still can't merge even now. Can you do it?

@bsipocz bsipocz merged commit debfeda into Caltech-IPAC:main May 15, 2025
6 of 8 checks passed
@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented May 15, 2025

Ahh, indeed, I enabled branch protection 😊

github-actions Bot pushed a commit that referenced this pull request May 15, 2025
Euclid MER catalog CMD: Reverse y-axis and zoom in debfeda
@troyraen troyraen deleted the raen/patch/euclid-mer-catalog branch May 16, 2025 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Content related issues/PRs. maintenance General maintenance of the content and/or infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants