-
Notifications
You must be signed in to change notification settings - Fork 30
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 reporting for llama testing and improve error handling #322
Conversation
@@ -256,3 +256,23 @@ def get_iree_flags(request: FixtureRequest): | |||
model_path["iree_hal_target_backends"] = set_fixture_from_cli_option( | |||
request, "--iree-hal-target-backends", "iree_hal_target_backends" | |||
) | |||
|
|||
|
|||
def pytest_html_results_table_header(cells): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you use these functions anywhere?
Nit: Maybe a small comment to what the functions are doing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment. Yeah, these functions allow us to customize the html reports and add the XFail Reason column in the https://nod-ai.github.io/SHARK-Platform.
- name: Deploy to GitHub Pages | ||
uses: peaceiris/actions-gh-pages@v3 | ||
with: | ||
github_token: ${{ secrets.SHARK_PLATFORM_GH_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This broke the CI for PRs from forks, see #395. Sending a PR to deactivate this until fixed.
This commit generates reports for the llama testing. Every time it runs it in CI, it will deploy to github pages (https://nod-ai.github.io/SHARK-Platform). @aviator19941 is working on getting the rest of the tests properly implemented. Only the llama8b fp16 tests are working as intended at the moment.