Skip to content

Register kernel for Greater int64 - #1546

Merged
Yufeng Li (yufenglee) merged 3 commits into
masterfrom
roli/greater
Aug 2, 2019
Merged

Register kernel for Greater int64#1546
Yufeng Li (yufenglee) merged 3 commits into
masterfrom
roli/greater

Conversation

@yufenglee

Copy link
Copy Markdown
Member

Description: Describe your changes.
Register kernel for Greater int64
Motivation and Context

  • Why is this change required? What problem does it solve?
    We have models that need Greater int64
  • If it fixes an open issue, please link to the issue here.

@yufenglee
Yufeng Li (yufenglee) requested a review from a team as a code owner August 1, 2019 22:25
@yufenglee
Yufeng Li (yufenglee) merged commit a098be1 into master Aug 2, 2019
@snnn
Changming Sun (snnn) deleted the roli/greater branch August 2, 2019 22:05
Dmitri Smirnov (yuslepukhin) pushed a commit that referenced this pull request Mar 17, 2026
## Fix graph vertex ordering during topological ordering

Previous implementation returned vertices in reverse order even when
there are no dependencies in the graph. Iterate the vertices in reverse
order so input order can be retained.

## 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.

2 participants