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

PyArrow upgrade to 14.0.1 #881

Closed
wants to merge 5 commits into from

Conversation

cshaley
Copy link

@cshaley cshaley commented Dec 5, 2023

Issue: #862

Notes:

  • upgraded pyarrow to 14.0.1
  • added fsspec because 2 unit tests did not work without it
  • increased evidently version so we get a new version published with new pyarrow.

Questions:

  • do I need to increase the pyarrow version in the integrations folders' requirements.txt file?
  • do you like pinning with pyarrow<15? Or want a different pin?

@emeli-dral
Copy link
Contributor

Hi @cshaley ,
Thank you a lot for addressing this issue and making the contribution!

I apologise for the late reply. We discussed internally how to address this issue in a best way, and we were choosing between the approach you suggested and getting rid of pyarrow as dependency.

At this point, we decided to remove pyarrow from the dependencies and because of this we won't merge this PR.

I hope it solves the issue for you (the update is available in Evidently 0.4.11)

@emeli-dral emeli-dral closed this Dec 14, 2023
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.

None yet

2 participants