Skip to content
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

Print xarray objects in user guide #476

Closed
wants to merge 1 commit into from
Closed

Print xarray objects in user guide #476

wants to merge 1 commit into from

Conversation

santisoler
Copy link
Member

Print xarray.DataArray and xarray.Dataset objects in the user guide pages instead of display them and letting Jupyter to render them as HTML objects. This is a patch to output cells not rendering properly on Sphinx dark mode.

Relevant issues/PRs:

This intends to patch the xarray issue mentioned in #472

Print `xarray.DataArray` and `xarray.Dataset` objects in the user guide
pages instead of display them and letting Jupyter to render them as HTML
objects. This is a patch to output cells not rendering properly on
Sphinx dark mode.
@leouieda
Copy link
Member

This is a known issue in xarray: pydata/xarray#8097 (comment)

I think we could just wait for a fix on their part. What do you think?

@santisoler
Copy link
Member Author

It's fine for me, @leouieda. I think we can go back to this PR before making the next Harmonica release and see if the problem was solved upstream. Otherwise we can apply this quick fix, release, and revert it in a future, when the displays look nice on dark theme. Does it make sense?

@santisoler
Copy link
Member Author

I'm closing this PR for now, but I'm not deleting the branch in case we need to get these changes back.

@santisoler santisoler closed this Mar 22, 2024
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.

None yet

2 participants