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

Filter matplotlib-related warnings #2344

Merged
merged 7 commits into from
Mar 2, 2023
Merged

Conversation

rjgildea
Copy link
Contributor

See matplotlib/matplotlib#25244 and matplotlib/matplotlib#25204

Consolidate filterwarnings in pytest.ini

Copy link
Member

@ndevenish ndevenish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a good solution. TIL capsys fixture.

@codecov
Copy link

codecov bot commented Feb 20, 2023

Codecov Report

Merging #2344 (07edc8b) into main (1d5db64) will decrease coverage by 0.24%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main    #2344      +/-   ##
==========================================
- Coverage   83.11%   82.88%   -0.24%     
==========================================
  Files         593      593              
  Lines       68596    68591       -5     
  Branches     9211     9211              
==========================================
- Hits        57013    56849     -164     
- Misses       9452     9627     +175     
+ Partials     2131     2115      -16     

@rjgildea rjgildea merged commit ab659cb into main Mar 2, 2023
@rjgildea rjgildea deleted the disable-matplotlib-warnings branch March 2, 2023 20:18
rjgildea added a commit to regro-cf-autotick-bot/dials-feedstock that referenced this pull request Mar 2, 2023
rjgildea added a commit to conda-forge/dials-feedstock that referenced this pull request Mar 7, 2023
* updated v3.13.0

* Remote unnecessary patch, unrestrict python

Conda-forge only does python >3.7 now

* Not like that.

* Use dxtbx 3.13

* Remove boost pins

* Patch test_ssx_integrate

See dials/dials#2344

* See which patches are still required

* Revert "See which patches are still required"

This reverts commit 9c54dda.

* Only remove 3/4 patches

* Patch for dials/dials#2356

* Only patch the test file

* Revert "Only patch the test file"

This reverts commit 9feb68f.

* Revert "Patch for dials/dials#2356"

This reverts commit 5e2deac.

* dev.dials.ssx* -> dials.ssx*

* CMake: Check for snprintf on windows

See dials/dials#2357
dials/dials@3fdbd25

* Add dev.dials.simple_integrate entry point

* Run single failing test to see if it still fails

* Now enable all tests but call dials.data before pytest

* Set nproc=1 when running rs_mapper tests

* Re-enable non-windows builds

* Re-enable other windows builds

* Try again pre-downloading image_examples

---------

Co-authored-by: Nicholas Devenish <ndevenish@gmail.com>
Co-authored-by: Richard Gildea <richard.gildea@diamond.ac.uk>
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

4 participants