search

allenai / papermage

library supporting NLP and CV research on scientific papers
https://papermage.org
Apache License 2.0
665 stars 52 forks source link

Remove Restriction on Pandas Version #71

Closed mdr223 closed 6 months ago

mdr223 commented 6 months ago

Hello there!

I noticed that papermage currently restricts the version of pandas to be pandas<2, however I don't believe the project explicitly imports and uses the package. This is a bit of a challenge for developers using this project because a lot of other projects are upgrading to pandas>=2 to make use of new functionalities, which causes issues in dependency resolution.

I created a fork in which I removed the restriction, ran the CI pipeline, and found that the builds for Python 3.8, 3.9, and 3.11 all passed. As such -- unless there is a good reason for keeping the version at pandas<2 (which there very might well be) -- I was wondering if you would be willing to remove the restriction to make it easier for downstream developers to incorporate your awesome work into our project(s)!

mdr223 commented 6 months ago

Sure thing! I just made a change to bump the version