Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix removal of consecutive tranposes with branches #162

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jicampos
Copy link
Contributor

This PR adds logic to handle and remove consecutive Transpose operations with branches. Originally, the package only removed consecutive Transpose operations when no branches were involved.

  • Added a check to ensure correct removal of redundant Transpose nodes without affecting graph structure in RemoveConsecutiveChanFirstAndChanLastTrafos.
  • Additionally, a check has been added to prevent redundant Transpose operations in MoveChanFirstDownstream when handling forked branches in the graph.

@jicampos jicampos requested a review from maltanar December 17, 2024 17:58
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.

1 participant