Skip to content

ENH: adding euclid clusters tutorial#284

Draft
bsipocz wants to merge 8 commits intoCaltech-IPAC:mainfrom
bsipocz:ENH_euclid_clusters
Draft

ENH: adding euclid clusters tutorial#284
bsipocz wants to merge 8 commits intoCaltech-IPAC:mainfrom
bsipocz:ENH_euclid_clusters

Conversation

@bsipocz
Copy link
Member

@bsipocz bsipocz commented Mar 11, 2026

This is a draft PR for now as the notebook runs way too long as is (2+hr locally)

TODO (that came up during making this PR, we may have other things left from the initial review):

  • fix figure captions
  • add metadata footer
  • make sure it runs and runs well within the confines of CI
  • cut up code cells, they are way too long and with little narrative in between

Extra things that are definitely not merging blockers, but what would make an improvement:

  • paper parsing for the table could be done with astropy, that may eliminate the need of introducing a new dependency (e.g. currently the notebook is failing due to the missing lxml library)
  • consider not using seaborn, as it is now a new dependency. If we were to use it in other notebooks, too then that's fine, but if it will be only this one, then I think we should reconsider to stay with mpl.

Resolves #282

Closes #283

@bsipocz bsipocz added the content: euclid Content related issues/PRs for notebooks with Euclid relevance label Mar 11, 2026
@jkrick
Copy link
Contributor

jkrick commented Mar 11, 2026

@bsipocz do you want me to look at how to shorten the runtime of this notebook? and also do you want me to look into adding narrative text? or do you already have a plan?

@bsipocz
Copy link
Member Author

bsipocz commented Mar 11, 2026

I would appreciate any help, thanks for offering.

@bsipocz bsipocz marked this pull request as draft March 11, 2026 22:38
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bsipocz
Copy link
Member Author

bsipocz commented Mar 12, 2026

@jkrick - you should be able to directly push to this branch. Based on what you showed today I think will know how to do that, but please ping me if you need any help.

@bsipocz
Copy link
Member Author

bsipocz commented Mar 12, 2026

Runtime actually doesn't look as bad on CI, I'm not sure why I had more issues locally. That being said if there is a reasonable way, maybe we could still cut it back to ~half?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content: euclid Content related issues/PRs for notebooks with Euclid relevance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add description of Euclid clusters notebook Release Euclid clusters notebook

3 participants