Skip to content

Conversation

@msoechting
Copy link
Collaborator

  • Added contents for the "Lexcube viewer" page
  • Added Lexcube example notebook to both docs/guide/juypterlab/notebooks/ and notebooks/

@AliceBalfanz
Copy link
Member

AliceBalfanz commented May 22, 2025

Hi @msoechting , did some settings prevent you from creating a branch in the repo directly? It makes my life a bit complicated that your PR is not on a branch within the repo but a fork. Would you mind creating the PR on a branch from within this repo? Would make testing on my side easier :)

@AliceBalfanz AliceBalfanz self-requested a review May 22, 2025 13:11
@msoechting
Copy link
Collaborator Author

Hi @AliceBalfanz, yes, it did not allow me to create a branch here. I assume because I am not an org member and not a collaborator. If you can arrange either of those, I can push my branch also in this repo 🙂

@AliceBalfanz
Copy link
Member

Hi Maximilian,

I have tested your nb in deepesdl. However it does not work for me. I have used this env:

channels:
  - conda-forge
dependencies:
  - python
  - xcube=1.11.0
  - xcube-cds
  - xcube-sh
  - xcube-cmems >= 0.1.6
  - xcube-cci
  - xcube-smos >= 0.3.1
  - xcube-zenodo
  - xcube-clms
  - xcube-stac >= 0.3.0
  - xcube_geodb
  - xrlint
  - boto3
  - cartopy
  - ipykernel
  - h5netcdf
  - rechunker
  - seaborn
  - zappend
  - gcsfs
  - pip
  - plotly
  - coiled
  - pyopenssl >= 23.2.0
  - pydap >= 3.4.0,<= 3.5.3
  - pip:
      - lexcube
      - ipywidgets
variables: {}

Is it possible that you invest time to test the nb in deepesdl and get the env correct and let us know the env that should be created for the nb? you can create your personal envs by using this guide:
https://earthsystemdatalab.net/guide/jupyterlab/#creating-custom-team-python-environment

@AliceBalfanz
Copy link
Member

Hi Max, we had another look at the lexcube NB and fail already at cell4:

image

could you have a look into the Cube3DWidget function?

Thanks!

@msoechting
Copy link
Collaborator Author

Hi Alicja, .plot has been added in a somewhat recent version, you may need to update the package. Sorry about not responding earlier to this issue - I did not have the opportunity to start using the DeepESDL platform and create a suitable environment myself.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants