Skip to content

remove np.all(... == ...) in test_surface.py#6218

Merged
melonora merged 1 commit intonapari:mainfrom
Carreau:more-np-array-equal-III
Sep 14, 2023
Merged

remove np.all(... == ...) in test_surface.py#6218
melonora merged 1 commit intonapari:mainfrom
Carreau:more-np-array-equal-III

Conversation

@Carreau
Copy link
Copy Markdown
Contributor

@Carreau Carreau commented Sep 11, 2023

See #6126 and #6213 among other.

Doing this in multiple PRs to make review easier.

@github-actions github-actions bot added the tests Something related to our tests label Sep 11, 2023
@Czaki Czaki added this to the 0.5.0 milestone Sep 11, 2023
@Czaki Czaki added the ready to merge Last chance for comments! Will be merged in ~24h label Sep 11, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 11, 2023

Codecov Report

Merging #6218 (b6d84d2) into main (e7ef939) will increase coverage by 0.04%.
Report is 4 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6218      +/-   ##
==========================================
+ Coverage   91.63%   91.68%   +0.04%     
==========================================
  Files         583      583              
  Lines       51124    51203      +79     
==========================================
+ Hits        46850    46945      +95     
+ Misses       4274     4258      -16     
Files Changed Coverage Δ
napari/layers/surface/_tests/test_surface.py 100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

@Czaki Czaki added the maintenance PR with maintance changes, label Sep 11, 2023
@melonora melonora merged commit 9bc71c5 into napari:main Sep 14, 2023
@melonora melonora removed the ready to merge Last chance for comments! Will be merged in ~24h label Sep 14, 2023
kne42 added a commit to kne42/napari that referenced this pull request Sep 19, 2023
* main: (26 commits)
  Fix some typing in napari.components (napari#6203)
  Use class name for object that does not have qt name (napari#6222)
  test: [Automatic] Constraints upgrades: `hypothesis`, `magicgui`, `psygnal`, `tensorstore`, `tifffile`, `tqdm`, `virtualenv` (napari#6143)
  Replace more np.all( ... = ...) with np.array_equal (napari#6213)
  remove np.all(... == ...) in test_surface.py (napari#6218)
  Ensure pandas Series is initialized with a list as data (napari#6226)
  Stop using temporary directory for store array for paint test (napari#6191)
  Bugfix: ensure thumbnail represents canvas when multiscale (napari#6200)
  cleanup np.all(... == ...) from test_points.py (napari#6217)
  [pre-commit.ci] pre-commit autoupdate (napari#6221)
  use app-model for file menu (napari#4865)
  Add tests to cover slicing behavior when changing layers and data (napari#4819)
  [pre-commit.ci] pre-commit autoupdate (napari#6128)
  Add test coverage for async slicing of labels (napari#5325)
  Add collision check when set colors for labels layer (napari#6193)
  Update "toggle ndview" text (napari#6192)
  Prevent layer controls buttons changing layout while taking screenshots with flash effect on (napari#6194)
  Fix typing in napari.utils.perf (napari#6132)
  Add GUI test coverage for changes to Labels.show_selected_label (napari#5372)
  Fix types in 'napari.utils.colormaps.categorical_colormap' (napari#6154)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance PR with maintance changes, tests Something related to our tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants