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

pandas deprecation warning #232

Merged
merged 1 commit into from
Jul 22, 2020
Merged

pandas deprecation warning #232

merged 1 commit into from
Jul 22, 2020

Conversation

rouille
Copy link
Collaborator

@rouille rouille commented Jul 17, 2020

Purpose

Update pandas set_option to avoid deprecation warning message

What is the code doing?

There is no code. I got the following message when running the test:

======================================================================= warnings summary =======================================================================
powersimdata/scenario/scenario.py:15
  /Users/brdo/CEM/PowerSimData/powersimdata/scenario/scenario.py:15: FutureWarning: Passing a negative integer is deprecated in version 1.0 and will not be supported in future version. Instead, use None to not limit the column width.
    pd.set_option("display.max_colwidth", -1)

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======================================================== 185 passed, 10 deselected, 1 warning in 25.28s ========================================================

I have hence replaced the value of the second argument as suggested in the message above.

Where to look

The option is set in the powersimdata.scenario.scenario module

Time estimate

5 min. Run the test using purest . -m "not integration" in the develop branch first, checkout the ben/pandas branch and run the test again

@rouille rouille added this to the The Last Dance milestone Jul 17, 2020
@rouille rouille merged commit 7a0ae71 into develop Jul 22, 2020
@rouille rouille deleted the ben/pandas branch July 22, 2020 16:35
@ahurli ahurli mentioned this pull request Mar 11, 2021
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 this pull request may close these issues.

2 participants