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

Remove a redundant XMR test; Add weights_only argument for torch.load #303

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

hallogameboy
Copy link
Contributor

@hallogameboy hallogameboy commented Jan 31, 2025

Issue #, if available:

Description of changes:

  • Remove a redundant test that requires pulling external pre-trained models.
  • Add the argumentweight_only=False for torch.load due to the change of default argument in PyTorch of later versions.
  • Change the PyTest workflow of aarch64 to use ubuntu-22.04-arm instead of ubuntu-latest to avoid docker pulling errors.
  • Add the include of cstdint at pecos/core/utils/file_util.hpp to avoid CodeQL errors.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@hallogameboy hallogameboy force-pushed the remove-redundant-branch branch 18 times, most recently from b6add21 to 255b95e Compare January 31, 2025 22:27
@hallogameboy hallogameboy force-pushed the remove-redundant-branch branch from 255b95e to a4e26f1 Compare January 31, 2025 22:35
@vpung vpung self-requested a review January 31, 2025 22:54
@hallogameboy hallogameboy merged commit 71adbed into amzn:mainline Jan 31, 2025
25 checks passed
@hallogameboy hallogameboy deleted the remove-redundant-branch branch January 31, 2025 22:55
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