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

Test of num_regression user message failing #3

Closed
nicoddemus opened this issue Jul 13, 2018 · 6 comments
Closed

Test of num_regression user message failing #3

nicoddemus opened this issue Jul 13, 2018 · 6 comments
Assignees

Comments

@nicoddemus
Copy link
Member

tests/test_num_regression.py:test_common_cases currently fails when running from a virtual environment on Windows because it produces this message:

data1:
          obtained_data1         ...                          diff
500  1.20000000000000018         ...           0.10000000000000009

While the expected was this one:

data1:
          obtained_data1       expected_data1                 diff
500  1.20000000000000018  1.10000000000000009  0.10000000000000009

It seems there's some issue with how pandas formats the tables.

The weird thing is that the same test passes when run inside our conda environment.

The test has been marked as xfail for now.

nicoddemus added a commit to nicoddemus/pytest-regressions that referenced this issue Jul 13, 2018
One of the tests need investigation (ESSS#3)
@marcelotrevisani
Copy link

I think I know how to fix it
Let's see

@marcelotrevisani
Copy link

Did you merge the branch with this test tests/test_num_regression.py:test_common_cases ?

@nicoddemus
Copy link
Member Author

Oh that's cool, wait up I will open the PR.

@nicoddemus nicoddemus mentioned this issue Jul 13, 2018
6 tasks
@nicoddemus
Copy link
Member Author

Please see #4

@tadeu
Copy link
Member

tadeu commented Jul 27, 2018

Might have to do with different terminal width, check this: https://stackoverflow.com/questions/11707586/python-pandas-how-to-widen-output-display-to-see-more-columns

A configuration like pd.set_option('display.width', 1000) might help.

@nicoddemus
Copy link
Member Author

It seems someone found a solution for a similar problem: pytest-dev/pytest#4030 (comment)

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

No branches or pull requests

3 participants