roboflow / inference

A fast, easy-to-use, production-ready inference server for computer vision supporting deployment of many popular model architectures and fine-tuned models.

https://inference.roboflow.com

Other

1.3k stars 116 forks source link

Loose boto3 requirements #457

Closed iurisilvio closed 3 months ago

iurisilvio commented 3 months ago

Description

Bump the max boto3 version to avoid pip trying to install many versions before finding one that works.

It is reasonably safe, boto3 is very stable.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How has this change been tested, please provide a testcase or example of how you tested the change?

🟢 CI pass 🟢

PawelPeczek-Roboflow commented 3 months ago

Does it solve the problem with aws-cli?

iurisilvio commented 3 months ago

@PawelPeczek-Roboflow yes! You can see here it uses the first version found.

https://github.com/roboflow/inference/actions/runs/9416030487/job/25938277946#step:4:136

PawelPeczek-Roboflow commented 3 months ago

amazing

PawelPeczek-Roboflow commented 3 months ago

U are free to merge once those run without issues: https://github.com/roboflow/inference/actions/runs/9462718021 https://github.com/roboflow/inference/actions/runs/9462730758

PawelPeczek-Roboflow commented 3 months ago

all 🟢 - u are free to merge