-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rename and docs update rebase #91
Conversation
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 know Abhi walked through everything, so let's go ahead and merge.
@annehaley Can you take a look once more and approve if this looks okay? |
pyproject.toml
Outdated
all = [ | ||
# viewer | ||
"trame>=3.6", | ||
"trame-vtk>=2.6", | ||
"trame-vuetify>=2.4", | ||
"geovista>=0.4", | ||
|
||
# esgf | ||
"intake-esgf>=2024.1", | ||
|
||
# pangeo | ||
"intake==0.7.0", # latest has plugin errors | ||
"intake-xarray>=0.7", | ||
"intake-esm>=2023.11", | ||
"s3fs>=2024.2", | ||
"gcsfs>=2024.2", |
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 think this change is outdated, so you can take this out. I believe @johnkit had mentioned that the recursive dependency definition doesn't work in some cases, so we should use the explicit list.
c8bf5f8
to
321f9bb
Compare
…drawer changes state
321f9bb
to
cdf7e21
Compare
This is a duplicate of MR #83
It contains the changes for renaming
pan3D-viewer
togeotrame
and updating documentation wherever necessaryI created the MR from a new branch so that we do not lose Anne's branch in case I messed up the rebase.