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

get rid of the sources of warnings in tests #326

Closed
aaronspring opened this issue Feb 27, 2020 · 3 comments · Fixed by #333
Closed

get rid of the sources of warnings in tests #326

aaronspring opened this issue Feb 27, 2020 · 3 comments · Fixed by #333
Assignees

Comments

@aaronspring
Copy link
Collaborator

You are encouraged to add information about all relevant sections, but not required to submit an issue.

Code Sample, a copy-pastable example if possible

Problem description

This should explain why the current behaviour is a problem and why the expected output is a better solution.

Expected Output

Output of climpred.show_versions()

# Paste the output here xr.show_versions() here
@aaronspring
Copy link
Collaborator Author

gazillions of warnings are back

@aaronspring aaronspring reopened this Jul 30, 2020
@aaronspring aaronspring pinned this issue Jul 30, 2020
@aaronspring aaronspring mentioned this issue Sep 9, 2020
25 tasks
@aaronspring aaronspring unpinned this issue Sep 17, 2020
@bradyrx
Copy link
Collaborator

bradyrx commented Sep 18, 2020

I feel like we are always chasing this. But good job catching it. Will also need to think about this for all the docs warnings.

@bradyrx
Copy link
Collaborator

bradyrx commented Sep 18, 2020

Addressed in #441. Let me know if that's true..

@bradyrx bradyrx closed this as completed Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants