-
Notifications
You must be signed in to change notification settings - Fork 7
Use TopologicalGraphService instead TopGraph #2092
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
Conversation
Test Results131 tests +1 131 ✅ +1 34s ⏱️ +14s Results for commit 08c9c75. ± Comparison against base commit f6efdd7. This pull request removes 9 and adds 10 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
| final TopPoint point, final Edge existingEdge, | ||
| final Boolean inTopDirection) { | ||
| final Node topDirNode = addPointGraphNode(point); | ||
| final Node againstTopDirNode = addPointGraphNode(point); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The againstTopDirNode will not added to the Graph, because it have same position with topDirNode
...pmodel.extensions/src/org/eclipse/set/ppmodel/extensions/PunktObjektTopKanteExtensions.xtend
Show resolved
Hide resolved
...lipse.set.application/src/org/eclipse/set/application/graph/TopologicalGraphServiceImpl.java
Outdated
Show resolved
Hide resolved
java/bundles/org.eclipse.set.utils/src/org/eclipse/set/utils/graph/AsSplitTopGraph.java
Show resolved
Hide resolved
|
@TruongQuangSB And we have table differences now in the plaz_mode because we get other lengths for the berhöhungslinien. Can you please have a look and verify that they are correct? |
Use
TopologicalGraphServiceinstead of TopGraph/DiGraph at some point:Punkt_Objektwith topological direction