-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
TST: consolidate version checks, remove outdated #5157
Conversation
statsmodels/compat/pandas.py
Outdated
@@ -7,6 +7,8 @@ | |||
|
|||
|
|||
version = LooseVersion(pandas.__version__) | |||
pandas_lt_0_19_2 = version < '0.19.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This matches the existing code, but there is a 0.19.2 vs 0.19.1 mismatch. Please advise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surely should be lte and <= '0.19.2'
Related: #5036 |
Codecov Report
@@ Coverage Diff @@
## master #5157 +/- ##
=========================================
Coverage ? 81.55%
=========================================
Files ? 569
Lines ? 89610
Branches ? 10027
=========================================
Hits ? 73078
Misses ? 14265
Partials ? 2267
Continue to review full report at Codecov.
|
LGTM |
@josef-pkt defacto green, OKed by bashtage, and no merge conflict |
@josef-pkt merge conflicts are going to start piling up before too long... |
50542b1
to
90171e3
Compare
90171e3
to
27533b3
Compare
@josef-pkt this consolidation should definitely happen before 0.10.0 |
@josef-pkt any thoughts here? |
@josef-pkt let's do version-checking right |
@ChadFulton prevent future complications |
Use <= rather than <
Closes #1063.
We recently removed some version checks for unsupported numpy/scipy/pandas versions, but missed a few. This gets most or all of the remaining ones.
Consolidates version checks in the compat files so that in the future they are all in one place when it comes time to update the minimum required versions.