[ONNX]Fix export of flatten #40418
[ONNX]Fix export of flatten #40418
Conversation
💊 CI failures summary and remediationsAs of commit b424b4e (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 1 failure confirmed as flaky and can be ignored:
This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 66 times. |
zou3519
added
the
triaged
BowenBao
left a comment
BowenBao
left a comment
There was a problem hiding this comment.
please check ci failures, are they related to setting slice2 as -1?
|
@houseroad CI failures are not caused by this PR. Can you please take a look? Thanks |
|
@houseroad CI is green! can you please take a look? Thanks! |
|
@houseroad Can you please take a look? |
facebook-github-bot
left a comment
facebook-github-bot
left a comment
There was a problem hiding this comment.
@houseroad has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
@houseroad merged this pull request in 9b0393f. |
1 similar comment
|
@houseroad merged this pull request in 9b0393f. |
8 participants
Shape is passed to _reshape_to_tensor as a Constant and cannot infer shape of the input when model is exported with dynamic axes set. Instead of a Constant pass output of a subgraph Shape-Slice-Concat to compute the shape for the Reshape node in _reshape_to_tensor function.