Skip to content

Fix ORTSRV nightly build#1440

Merged
jywu-msft merged 3 commits into
microsoft:masterfrom
NonStatic2014:bohu/fix_nightly_build
Jul 22, 2019
Merged

Fix ORTSRV nightly build#1440
jywu-msft merged 3 commits into
microsoft:masterfrom
NonStatic2014:bohu/fix_nightly_build

Conversation

@NonStatic2014
Copy link
Copy Markdown
Contributor

Description: Nightly build failed due to could not find the model test path.

Motivation and Context

  • Why is this change required? What problem does it solve?
    The server_run_build.sh was created before merged back to master. When pick it up for GRPC build, the build_dir was not been updated. It will not impact the regular build since it self-sufficient for unit tests and integration tests. It is not enough for nightly build since nightly build depends on the model downloaded from blob storage.

  • If it fixes an open issue, please link to the issue here.

@NonStatic2014 NonStatic2014 requested a review from a team as a code owner July 19, 2019 17:03
jywu-msft
jywu-msft previously approved these changes Jul 19, 2019
@jywu-msft
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 22 pipeline(s).

@pranavsharma
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 22 pipeline(s).

@jywu-msft jywu-msft merged commit 2277341 into microsoft:master Jul 22, 2019
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
…lerators (#1440)

## Update olive_config and auto-opt to use precision, providers and
accelerators

* Implement providers, accelerators and precisions for pass in
olive_config.json.
Each pass module config now also requires listing supported providers,
accelerators
  and precisions.
* Update auto-opt cli to use a pre-defined order of passes when search
is disabled.
The list of passes is selected based on the user's choice of precision
for
  the output model.

## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [x] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [x] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.
- [ ] Is this PR including examples changes? If yes, please remember to
update [example
documentation](https://github.com/microsoft/Olive/blob/main/docs/source/examples.md)
in a follow-up PR.

## (Optional) Issue link
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