Skip to content

Update release branch directly to remove special casing of "None" as a dim_param - #1495

Closed
skottmckay wants to merge 1 commit into
rel-0.5.0from
skottmckay/rel-0.5.0/RemoveSpecialCasingOfNoneAsDimParam
Closed

Update release branch directly to remove special casing of "None" as a dim_param#1495
skottmckay wants to merge 1 commit into
rel-0.5.0from
skottmckay/rel-0.5.0/RemoveSpecialCasingOfNoneAsDimParam

Conversation

@skottmckay

Copy link
Copy Markdown
Contributor

Description:
Duplicate changes from #1482 on release branch

Motivation and Context
CI builds are extremely flaky currently and this change needs to get into the release asap. Duplicate the change so the release update isn't blocked on #1482 being commited on master and cherry-picked across to the release branch.

@skottmckay
skottmckay requested a review from a team as a code owner July 25, 2019 05:37
@skottmckay
skottmckay requested a review from hariharans29 July 25, 2019 05:37
@yuslepukhin
yuslepukhin deleted the skottmckay/rel-0.5.0/RemoveSpecialCasingOfNoneAsDimParam branch August 6, 2019 17:03
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
## Describe your changes
Fix RUFF format
## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [ ] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [ ] 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.

4 participants