Skip to content

Conversation

@Erol444
Copy link
Contributor

@Erol444 Erol444 commented Mar 4, 2022

No description provided.

Copy link
Collaborator

@themarpe themarpe left a comment

Choose a reason for hiding this comment

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

Regardless of the comments (+ we could most likely get rid of the anaconda here...), should probs merge, we'll update once better steps come along as we go...

Comment on lines +103 to +104
# Remove code of block (3 lines) starting with: if thisPlatform == "arm64" and platform.system() == "Darwin":
# Remove code of block (48 lines) starting with: if not args.skip_depthai:
Copy link
Collaborator

@themarpe themarpe Mar 7, 2022

Choose a reason for hiding this comment

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

We should get the fix in instead of this...
(got a branch for this, will open a PR tomorrow)

EDIT: I'll address when GitHub M1 Runner support is out...

@themarpe themarpe merged commit 3a6e007 into main Mar 8, 2022
@themarpe themarpe deleted the m1_build_wheels_tutorial branch March 8, 2022 13:14
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.

3 participants