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

Feature / Updated transformer ranking #698

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Jerry-Jinfeng-Guo
Copy link
Contributor

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo commented Aug 20, 2024

This PR implements the updated transformer ranking in #662

  • Unregulated transformer edges get weight 0 instead of 1
  • Update unit tests
    • Add unit tests with un-regulated transformers
    • Update existing tests based on new correct results
    • Update docs

Signed-off-by: Jerry Guo <[email protected]>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added feature New feature or request improvement Improvement on internal implementation labels Aug 20, 2024
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Aug 20, 2024
Copy link

sonarcloud bot commented Aug 20, 2024

Copy link

sonarcloud bot commented Sep 24, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request improvement Improvement on internal implementation
Projects
Status: Q3 2024
Development

Successfully merging this pull request may close these issues.

1 participant