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

Setting pandas!=2.1.0 #1267

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Setting pandas!=2.1.0 #1267

merged 2 commits into from
Sep 11, 2023

Conversation

andrewlee94
Copy link
Member

@andrewlee94 andrewlee94 commented Sep 11, 2023

Fixes #1253

Summary/Motivation:

Pandas introduced a bug in v2.1.0 that is causing Pysmo tests to fail. It looks like this bug will be fixed in the upcoming v2.1.1 pandas release.

Changes proposed in this PR:

  • Setting "pandas!=2.1.0" in setup.py to avoid using the buggy version of pandas and to allow us to switch the the corrected version once it is released.

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@andrewlee94 andrewlee94 self-assigned this Sep 11, 2023
@andrewlee94 andrewlee94 added bug Something isn't working Priority:Normal Normal Priority Issue or PR dependencies Pull requests that update a dependency file surrogates PySMO labels Sep 11, 2023
@andrewlee94 andrewlee94 changed the title Setting pandas!=2.1 Setting pandas!=2.1.0 Sep 11, 2023
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (02c20ac) 76.73% compared to head (818a31b) 76.72%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1267      +/-   ##
==========================================
- Coverage   76.73%   76.72%   -0.01%     
==========================================
  Files         382      382              
  Lines       61232    61232              
  Branches    11297    11297              
==========================================
- Hits        46985    46982       -3     
- Misses      11813    11817       +4     
+ Partials     2434     2433       -1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lbianchi-lbl lbianchi-lbl added the CI:run-integration triggers_workflow: Integration label Sep 11, 2023
@idaes-build idaes-build removed the CI:run-integration triggers_workflow: Integration label Sep 11, 2023
@andrewlee94 andrewlee94 enabled auto-merge (squash) September 11, 2023 15:15
@andrewlee94 andrewlee94 merged commit ab29c81 into IDAES:main Sep 11, 2023
58 of 66 checks passed
@andrewlee94 andrewlee94 deleted the pandas_bug branch September 11, 2023 16:04
lbianchi-lbl pushed a commit that referenced this pull request Sep 14, 2023
* Setting pandas!=2.1

* Specificying minor release to exclude

(cherry picked from commit ab29c81)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file Priority:Normal Normal Priority Issue or PR PySMO surrogates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pysmo tests np.testing.assert_array_equal() fail because of ordering
4 participants