Skip to content

TST/COMPAT: test memory usage failed on macOS after change pytest-xdist -dist to no #39096

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

Open
fangchenli opened this issue Jan 10, 2021 · 2 comments
Labels
CI Continuous Integration Testing pandas testing functions or related to the test suite

Comments

@fangchenli
Copy link
Member

In #38905, I changed the distribution algorithm of pytest-xdist to no from loadfile. And the test pandas/tests/base/test_misc.py:test_memory_usage failed on macOS.

index_or_series_obj = Series([], Name: a, dtype: float64)

    @pytest.mark.skipif(PYPY, reason="not relevant for PyPy")
    def test_memory_usage(index_or_series_obj):
        obj = index_or_series_obj
    
        res = obj.memory_usage()
        res_deep = obj.memory_usage(deep=True)
    
        is_object = is_object_dtype(obj) or (
            isinstance(obj, Series) and is_object_dtype(obj.index)
        )
        is_categorical = is_categorical_dtype(obj.dtype) or (
            isinstance(obj, Series) and is_categorical_dtype(obj.index.dtype)
        )
    
        if len(obj) == 0:
            if isinstance(obj, Index):
                expected = 0
            else:
                expected = 108 if IS64 else 64
>           assert res_deep == res == expected
E           assert 0 == 108
@fangchenli fangchenli added Bug Needs Triage Issue that has not been reviewed by a pandas team member Compat pandas objects compatability with Numpy or Python functions Testing pandas testing functions or related to the test suite and removed Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Jan 10, 2021
@simonjayhawkins
Copy link
Member

failed on macOS.

same error on WSL with both n=auto and without specifying n

@mroeschke mroeschke added CI Continuous Integration and removed Compat pandas objects compatability with Numpy or Python functions labels Aug 15, 2021
@Apteryks
Copy link

Same on GNU+Linux when using pytest-xdist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Testing pandas testing functions or related to the test suite
Projects
None yet
Development

No branches or pull requests

4 participants