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.

Sign up for GitHub

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

Jump to bottom

Adds a layout unzipping intermediate processor after crossing minimization #1052

Merged
merged 6 commits into from
Jul 18, 2024
Merged

Adds a layout unzipping intermediate processor after crossing minimization #1052

merged 6 commits into from
Jul 18, 2024

Conversation

Eddykasp
Copy link
Contributor

@Eddykasp Eddykasp commented Jul 10, 2024 •
edited
Loading

TODO:

  • add tests
  • expand explanation of placeholder node types
  • improve explanation of what the layer unzipper does (alternating is not clear enough)
  • add option to reset 'shift ladder' when hitting long edge dummies
  • search for layer split property on all nodes of a layer and take the first one that is set, otherwise take the default value

@Eddykasp
Properties of layer unzipping
f333009 
Add property to control whether the alternating pattern resets after
long edges.

Search for any node in a layer that sets a property and apply the first
found to the layer of that node.

Move Properties from crossing minimization group to their own group.
@Eddykasp
Improve documentation of layer unzipper.
5066349
@Eddykasp
Add tests for n-layer unzipper.
1200047
@Eddykasp Eddykasp marked this pull request as ready for review July 15, 2024 15:18
@Eddykasp Eddykasp added this to the Release 0.9.2 milestone Jul 18, 2024
@Eddykasp Eddykasp merged commit ad04950 into eclipse:master Jul 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@soerendomroes soerendomroes soerendomroes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 0.9.2
Development

Successfully merging this pull request may close these issues.
2 participants
@Eddykasp @soerendomroes