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

PyTest-8 has changed keyword arguments #1670

Open
leftaroundabout opened this issue Dec 7, 2024 · 0 comments · May be fixed by #1672
Open

PyTest-8 has changed keyword arguments #1670

leftaroundabout opened this issue Dec 7, 2024 · 0 comments · May be fixed by #1672

Comments

@leftaroundabout
Copy link
Contributor

This occurs only for skipped Astra tests, with pytest==8.3.4 (and not with pytest==7.4.4):

________________________________________________________________________________________________________ test_anisotropic_voxels[ geometry='par2d' ] _________________________________________________________________________________________________________

geometry = Parallel2dGeometry(
    uniform_partition(0.0, 3.142, 100),
    uniform_partition(-30.0, 30.0, 100)
)

    def test_anisotropic_voxels(geometry):
        """Test projection and backprojection with anisotropic voxels."""
        ndim = geometry.ndim
        shape = [10] * (ndim - 1) + [5]
        space = odl.uniform_discr([-1] * ndim, [1] * ndim, shape=shape,
                                  dtype='float32')
    
        # If no implementation is available, skip
        if ndim == 2 and not odl.tomo.ASTRA_AVAILABLE:
>           pytest.skip(msg='ASTRA not available, skipping 2d test')
E           TypeError: skip() got an unexpected keyword argument 'msg'

pypitest.pyenv/lib/python3.12/site-packages/odl/test/tomo/operators/ray_trafo_test.py:362: TypeError
leftaroundabout added a commit that referenced this issue Dec 7, 2024
This is a workaround to a very small problem with pytest-8,
#1670
leftaroundabout added a commit to odlgroup/odl-feedstock that referenced this issue Dec 9, 2024
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 a pull request may close this issue.

1 participant