Skip to content

Commit

Permalink
Fix minor
Browse files Browse the repository at this point in the history
  • Loading branch information
goodsong81 committed Mar 19, 2024
1 parent 60ce92c commit 8593bd4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
9 changes: 2 additions & 7 deletions .github/workflows/perf_benchmark.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,7 @@ on:
Target OTX ref (tag / branch name / commit hash) on main repo to test. Defaults to the current branch.
`pip install otx[full]@https://github.com/openvinotoolkit/training_extensions.git@{otx_ref}` will be executed before run,
and reverted after run. Works only for v1.x assuming CLI compatibility.
artifact-prefix:
type: string
default: perf-benchmark
default: __CURRENT_BRANCH_COMMIT__
workflow_call:
inputs:
model-category:
Expand Down Expand Up @@ -90,9 +88,6 @@ on:
Target OTX ref (tag / branch name / commit hash) on main repo to test. Defaults to the current branch.
`pip install otx[full]@https://github.com/openvinotoolkit/training_extensions.git@{otx_ref}` will be executed before run,
and reverted after run. Works only for v1.x assuming CLI compatibility.
artifact-prefix:
type: string
default: perf-benchmark
# Declare default permissions as read only.
permissions: read-all
Expand Down Expand Up @@ -152,7 +147,7 @@ jobs:
- name: Upload test results
uses: actions/upload-artifact@5d5d22a31266ced268874388b861e4b58bb5c2f3 # v4.3.1
with:
name: ${{ inputs.artifact-prefix }}-${{ matrix.task-short }}
name: pef-benchmark-${{ matrix.task-short }}
path: .tox/perf-*.csv
# Use always() to always run this step to publish test results when there are test failures
if: ${{ always() }}
3 changes: 3 additions & 0 deletions tests/perf/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ def pytest_addoption(parser):
parser.addoption(
"--otx-ref",
type=str,
default="__CURRENT_BRANCH_COMMIT__",
help="Target OTX ref (tag / branch name / commit hash) on main repo to test. Defaults to the current branch. "
"`pip install otx[full]@https://github.com/openvinotoolkit/training_extensions.git@{otx_ref}` will be executed before run, "
"and reverted after run. Works only for v1.x assuming CLI compatibility.",
Expand All @@ -111,6 +112,8 @@ def fxt_current_date() -> str:
@pytest.fixture(scope="session")
def fxt_otx_ref(request: pytest.FixtureRequest) -> str | None:
otx_ref = request.config.getoption("--otx-ref")
if otx_ref == "__CURRENT_BRANCH_COMMIT__":
otx_ref = None

if otx_ref:
# Install specific version
Expand Down

0 comments on commit 8593bd4

Please sign in to comment.