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

added openvino and its dependencies #12

Closed
wants to merge 3 commits into from

Conversation

dazmashaly
Copy link
Contributor

No description provided.

added openvino
added openvino and its dependencies
@dazmashaly
Copy link
Contributor Author

most of the errors I'm getting are regarding Pytorch and TensorFlow and not the library I added, is this due to the library I just added, or is this normal?

@isms
Copy link
Contributor

isms commented Feb 29, 2024

@dazmashaly Yes, it is caused by libopenvino, or at least the version specified, c.f. here: https://github.com/drivendataorg/spacecraft-pose-object-detection-runtime/actions/runs/8103460504/job/22148102893?pr=12#step:4:786

How did you choose this version? Did you test with micromamba or with the Docker container locally and verify this version is compatible with the other dependencies?

@dazmashaly dazmashaly closed this Mar 1, 2024
Copy link
Contributor

@isms isms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a quick fix that will make this PR work if you still need the dependency (see comment)

@@ -31,3 +31,6 @@ dependencies:
- ultralytics=8.1.11
- tqdm=4.66.2
- xgboost=2.0.3
- openvino=2023.3.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dazmashaly If you remove the other two lines below this line and change the version so it is openvino=2023.2.0 then it should work.

Do you still need this dependency?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isms yes thank you, I was still trying to solve it.

isms added a commit that referenced this pull request Mar 2, 2024
Replaces #12
@isms isms mentioned this pull request Mar 2, 2024
isms added a commit that referenced this pull request Mar 2, 2024
Replaces #12
isms added a commit that referenced this pull request Mar 2, 2024
@isms
Copy link
Contributor

isms commented Mar 2, 2024

@dazmashaly Should be all set now from #13

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.

2 participants