Skip to content

Remove all perf checks in preparation for beta-2 update#114

Merged
hongsuh-aws merged 1 commit intomainfrom
hongsuh/remove-perf-checks
Dec 5, 2025
Merged

Remove all perf checks in preparation for beta-2 update#114
hongsuh-aws merged 1 commit intomainfrom
hongsuh/remove-perf-checks

Conversation

@hongsuh-aws
Copy link
Copy Markdown
Contributor

@hongsuh-aws hongsuh-aws commented Dec 5, 2025

Removing all perf checks due to upcoming beta-2 release, which will involve an overhaul of this repo.

Testing:

Please see detailed unit test requirements in the CONTRIBUTING.md

  • The change is covered by numeric check using nki.baremetal
  • The change is covered by performance benchmark test using nki.benchmark
  • The change is covered by end-to-end integration test

Pull Request Checklist

  • I have filled in all the required field in the template
  • I have tested locally that all the tests pass
  • By submitting this pull request, I confirm that my contribution is made under the terms of the MIT-0 license.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

Please download and review the generated API doc at https://github.com/aws-neuron/nki-samples/actions/runs/19976988981

@hongsuh-aws hongsuh-aws requested a review from tonyz0x0 December 5, 2025 21:52
Copy link
Copy Markdown
Contributor

@tonyz0x0 tonyz0x0 left a comment

Choose a reason for hiding this comment

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

LGTM

@hongsuh-aws hongsuh-aws changed the title Remove all perf checks Remove all perf checks in preparation for beta-2 update Dec 5, 2025
@hongsuh-aws hongsuh-aws merged commit fc5c69f into main Dec 5, 2025
4 of 7 checks passed
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