-
Notifications
You must be signed in to change notification settings - Fork 20
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
MAINT: cleanup rendering warnings #321
Changes from all commits
4f1f4b2
d0978ce
d2cd0d5
f7e01a8
52ae1bc
c643cc9
3d56d51
478f775
b104f04
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Starting with a clean environment, I had to make a few additions to this file. |
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -2,11 +2,12 @@ | |||||||
# beginning of the notebook, make sure the lists are consistent and only | ||||||||
# contain dependencies that are actually used in the notebook. | ||||||||
tqdm | ||||||||
numpy | ||||||||
numpy<2 # SOMPY incompatibility | ||||||||
scipy | ||||||||
pandas | ||||||||
pandas[parquet] | ||||||||
matplotlib | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I missed this one, thanks! will rather add it to fada9e7 |
||||||||
scikit-learn | ||||||||
scikit-image | ||||||||
astropy | ||||||||
umap-learn | ||||||||
git+https://github.com/sevamoo/SOMPY | ||||||||
|
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -2,12 +2,18 @@ | |||||||
# beginning of the notebook, make sure the lists are consistent and only | ||||||||
# contain dependencies that are actually used in the notebook. | ||||||||
numpy | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the science_demo environment, I had to also install
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It gets pulled without issues both locally and in CI, so I would rather not list it as we don't use numba directly. |
||||||||
pandas | ||||||||
pandas[parquet] | ||||||||
matplotlib | ||||||||
astropy | ||||||||
sktime | ||||||||
tqdm | ||||||||
googledrivedownloader | ||||||||
scikit-learn | ||||||||
acstools | ||||||||
## Optional indirect dependencies required by functionalities used in the notebook | ||||||||
# Required by functionality we use from acstools | ||||||||
scikit-image | ||||||||
# Required by functionality we use from sktime | ||||||||
numba | ||||||||
# Required for sensible progress bars | ||||||||
ipywidgets |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
<!-- #region --> | ||
# Explore Euclid Data | ||
*** | ||
|
||
|
||
## Learning Goals | ||
By the end of this tutorial, you will be able to: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've had this notebook running for about an hour and it's still not done. It has been stuck on the second cell in section '4) Repeating the above, this time with ZTF + WISE manifold' for most of the time. It hasn't crashed (
top
shows that the CPU is still in use), though there are a bunch of warnings. I don't know whether this is normal/expected or not.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't comment on that as I got stuck with the data download both locally and in CI, maybe open a separate issue for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I can confirm that I also timeout on that cell locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Opened #324