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

[MISC]: More careful consideration of different xarray options #98

Open
jthielen opened this issue Sep 16, 2022 · 0 comments
Open

[MISC]: More careful consideration of different xarray options #98

jthielen opened this issue Sep 16, 2022 · 0 comments
Milestone

Comments

@jthielen
Copy link
Collaborator

jthielen commented Sep 16, 2022

What is your issue?

Test expected results under different xarray options

In the spirit of improving the quality of our tests (xref #60), it would be nice to implement tests where different relevant xarray options are enabled (using set_options as a context manager). This would likely make it easier to catch issues like #96 .

Xarray options in issue reports

Not sure the best way to do this (bundle into xwrf.show_versions()? Add another copy-paste box to the issue template?), but it could help with debugging if we knew the state of xarray.get_options.

@jthielen jthielen added this to the v0.1 milestone Sep 16, 2022
@jthielen jthielen changed the title [MISC]: Test expected results under different xarray options [MISC]: More careful consideration of different xarray options Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant