Labelbox / labelbox-python

The data factory for next gen AI
https://labelbox.com
Apache License 2.0
111 stars 58 forks source link

Bump dateutil max version #1716

Closed colonelpanic8 closed 1 month ago

colonelpanic8 commented 2 months ago

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

All Submissions

New Feature Submissions

Changes to Core Features

adrian-chang commented 2 months ago

Hey @colonelpanic8

So we use rye for development, not poetry so the PR as is won't be able to be approved.

Please read this for further details (https://github.com/Labelbox/labelbox-python/blob/develop/CONTRIBUTING.md).

We'll take care of the dateutil bump, but what is the need for a head version of OpenCV? We're not aware of any use case of the SDK needing a head version.

colonelpanic8 commented 2 months ago

@adrian-chang yeah sorry, i forgot i had made this pr and i was using this pr to mess around with a git build.

the original change was just a bump tho:

https://github.com/Labelbox/labelbox-python/pull/1716/commits/071b3960580ab4889626337f49de8f9dfd5a4ed2

colonelpanic8 commented 2 months ago

I adjusted things so that my poetry stuff is in another branch.

adrian-chang commented 1 month ago

I'm going to merge this in ... given a library bump, our GHActions will have to be updated to address remote forks, thanks though!