ENH: adding euclid clusters tutorial#284
Conversation
|
@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? |
|
I would appreciate any help, thanks for offering. |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
@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. |
|
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? |
…, and downsampling for display. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
I sped up runtime with some low hanging fruit - by parallelizing the download and downsampling the images for display. Let me know if this isn't enough. I'd prefer to do a major text wrangling in a separate PR, so I will make a separate issue for that, and please don't wait for it in this PR. |
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):
Extra things that are definitely not merging blockers, but what would make an improvement:
lxmllibrary)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