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

Add aarch64 python 3.10 build support #103

Open
wants to merge 5 commits into
base: FixInstallRequirements
Choose a base branch
from

Conversation

yasdum
Copy link

@yasdum yasdum commented Jan 15, 2023

Add aarch64 python 3.10 build support

manoj7410 and others added 2 commits October 18, 2021 11:53
Adding code to download testdata for movenet_pose_estimation
@google-cla
Copy link

google-cla bot commented Jan 15, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@@ -76,7 +76,7 @@ def main():
raise ValueError('Only support uint8 input type.')

size = common.input_size(interpreter)
image = Image.open(args.input).convert('RGB').resize(size, Image.ANTIALIAS)
image = Image.open(args.input).convert('RGB').resize(size, Image.LANCZOS)
Copy link

@Namburger Namburger Mar 1, 2024

Choose a reason for hiding this comment

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

There're too many things going on in this one PR, changes like this should be in a separate pr.
Anyhow, I recently merged another pr that've done this.

ifeq ($(ARCH),arm)
CPU ?= darwin_arm64
DARWIN_CPU := darwin_arm64
else
CPU ?= darwin

Choose a reason for hiding this comment

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

Thanks @yasdum!
I'm not super familiar with MacOS, why is there a need for a darwin and a darwin_x86_64?

scott306lr and others added 2 commits March 1, 2024 08:53
There was an index error that occurs sometimes when running a custom edge tpu model. Disagreement between count and len(scores). Confirmed that this resolves the bug
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

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.

7 participants