PS:@dstansby I didn't see that there was ongoing work on a different PR because it wasn't linked to the issue. Feel free to treat this as stale for while.

Hi@dstansby ,

thanks a lot for taking the time to review this! I think I can definitely get those changes done myself. I'll be on vacation in the coming week so PR should come sometime in the second-to-next week.

Hi@dstansby ,

I added your suggestions from above and replaced the magicgui widgets with pyqt code as has been done in the scatter widget in the meantime. Also sorry for the messy commit history - let me know if I should put the changes on a clean branch and re-start the PR.

pre-commit.ci autofix

Hi@dstansby ,

I'm not sure what you mean by figure test? I did add the following to the tests. This adds two points layers to the viewer with two sets of features assigned to each. I check whether the figure in the widget changes upon changing the key for the histogram and whether the plot changes as it should when the selected layer is changed.

n_points=1000random_points=np.random.random((n_points,3))*10feature1=np.random.random(n_points)feature2=np.random.normal(size=n_points)viewer=make_napari_viewer()viewer.add_points(random_points,properties={"feature1":feature1,"feature2":feature2},name="points1",    )viewer.add_points(random_points,properties={"feature1":feature1,"feature2":feature2},name="points2",    )widget=FeaturesHistogramWidget(viewer)viewer.window.add_dock_widget(widget)# Check whether changing the selected key changes the plotwidget._set_axis_keys("feature1")fig1=deepcopy(widget.figure)widget._set_axis_keys("feature2")assert_figures_not_equal(widget.figure,fig1)# check whether selecting a different layer produces the same plotviewer.layers.selection.clear()viewer.layers.selection.add(viewer.layers[1])assert_figures_equal(widget.figure,fig1)

For a figure test, see e.g.,

This using thepytest-mpl plugin to compare the generated image with an image stored in the repository in this directory:https://github.com/matplotlib/napari-matplotlib/tree/main/src/napari_matplotlib/tests/baseline

For the figure test: So@pytest.mark.mpl_image_compare will compare the returned image toany image in thebaseline directory?

No, you'll need to generate the new test image - seehttps://github.com/matplotlib/pytest-mpl#usage for instructions.

@dstansby

No, you'll need to generate the new test image - seematplotlib/pytest-mpl#usage for instructions.

Ok, I added the baseline image, but for some reason, the generated baseline is not detected by the test...Sorry for the back and forth, I'm not so familiar with these testing schemes 🙈

Just as a sidenote: Setting everything up forpytest --mpl seemed quite complicated to me. It requires a few additional installation steps (pip install pyqt6 tinycss2) that did not seem evident to me from the.

No worries! If you don't mind, I'd be happy to pick this up and get the test working?

No worries! If you don't mind, I'd be happy to pick this up and get the test working?

Not at all! Would be cool though if it were documented somewhere for future contributions 👍

Looks like the test image was just in the wrong directory :) I also did a few other minor fixes, will merge this and do a new release of napari-matplotlib if the tests pass. Thanks a lot for the contribution, and sorry for how long it took me to merge it...

Looks like it's just code coverage failing