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

Move Matches nearby needs combine option being selected twice #281

Open
KonradHoeffner opened this issue Jul 29, 2020 · 7 comments
Open
Assignees
Labels
feature a new feature or enhancement of existing functionality
Milestone

Comments

@KonradHoeffner
Copy link
Collaborator

KonradHoeffner commented Jul 29, 2020

On Prof. Winter's machine, "move matches nearby" can only be used after the "combine matches" option is unselected and then selected again. However this problem could not be replicated on Konrad's machine, see video.
Investigate.

@KonradHoeffner KonradHoeffner added this to the 1.7 milestone Jul 29, 2020
@KonradHoeffner KonradHoeffner self-assigned this Jul 29, 2020
@KonradHoeffner
Copy link
Collaborator Author

Excerpt from AW:

Ich hatte mir zu operatives Informationsmanagement einen Star gemacht und dann bei Options die „combine Matches“ angehakt. Alles gut. Dann habe ich reset view gemacht und einen Star zu „Project“ gemacht. Jetzt ist die combine matches Option weiter angehakt, aber es werrden nur die matches von „operatives Informationsmanagement“ angezeigt, die schon vorher angezeigt waren. Wenn ich jetzt die Option aus- und dann wieder einschalte, wird alles richtig angezeigt. Wir haben hier wieder das Problem mit der neuen Sicht, wenn man einen Star oder ähnliches erzeugt. Das gehört einfach in eigene Fenster, die jeweils für sich ordentlich gemanagt werden müssen.

@KonradHoeffner
Copy link
Collaborator Author

KonradHoeffner commented Aug 11, 2020

Ah jetzt verstehe ich die Ursache: Im star wird bei combine nur das derzeit Sichtbare zusammengeführt und dadurch muss man es neu aktivieren. Genauso verhält es sich mit Filtern. Ich habe probiert, auch unsichtbares zu kombinieren aber dann werden aus einem Star heraus alle Kombinationsknoten angezeigt, auch wenn deren Inhaltsknoten alle unsichtbar sind. Daher habe ich das früher Verhalten wiederhergestellt und schiebe diese Issue zu Thomas' Bachelorarbeit, damit er das Problem berücksichtigen kann.

@KonradHoeffner KonradHoeffner modified the milestones: 1.7, tp-must Aug 11, 2020
@ThomasPause ThomasPause self-assigned this Sep 4, 2020
@ThomasPause ThomasPause modified the milestones: tp-must, tp-optional Sep 30, 2020
@KonradHoeffner KonradHoeffner added feature a new feature or enhancement of existing functionality and removed bug investigate labels Feb 2, 2021
@KonradHoeffner
Copy link
Collaborator Author

Ich könnte eine Option einbauen "star operations in new view" und wenn diese aktiv ist Star und ähnliche Operationen in einem neuen Tab erzeugen.

@KonradHoeffner
Copy link
Collaborator Author

KonradHoeffner commented Feb 2, 2021

Herausforderungen

  • Staroperationen auf Mehrfachselektionen werden mehrfach ausgeführt. Dies muss also davor geschaltet werden.
  • Star verwendet den "center"-Parameter, der dann aber zum alten Graph gehört und daher nicht mehr gültig ist.
  • showDoubleStar verwendet den "to"-Parameter, der dann aber zum alten Graph gehört und daher nicht mehr gültig ist. Außerdem this.getSource().

Graph.getSource(); muss also den Zustand vom alten Graph übernehmen und die Parameter müssen umgeschrieben werden auf den neuen Graph.

@KonradHoeffner
Copy link
Collaborator Author

KonradHoeffner commented Feb 2, 2021

Wir brauchen also eine "assimilate"-Funktion, die ein oder mehrere Knoten und/oder Kanten nimmt, und anhand der IDs dann versucht, diese Elemente in dem neuen Graphen zu finden. Alle davon, die gefunden werden, werden dann umgeschrieben. Diese Funktion muss auch angewendet werden auf Elemente, die im Graphen gespeichert werden, z.B. die Pfadquelle, und beim Kopieren übernommen werden.

Damit könnte man dann auch copy/paste verallgemeinern, da dort ja das gleiche Prinzip zugrunde liegt.

@KonradHoeffner
Copy link
Collaborator Author

Graph.getSource() zu übernehmen ist zu kompliziert und unintuitiv. Besser in den Pfadoperationen "from" und "to"-Parameter verwenden und die Quell- und Zielauswahl in den Aufruf auslagern.

@KonradHoeffner KonradHoeffner modified the milestones: 2.1, 2.2 Jun 16, 2021
@KonradHoeffner KonradHoeffner modified the milestones: 2.2, 3.1 Aug 25, 2021
@KonradHoeffner KonradHoeffner removed their assignment May 25, 2022
@KonradHoeffner KonradHoeffner modified the milestones: 22.05, next May 25, 2022
@Yagnap
Copy link
Collaborator

Yagnap commented Oct 9, 2024

Mir fällt gerade auf, dass ich so eine Funktion aus Versehen geschrieben habe, um copy paste zu fixen (#346), wobei assimilate im Nachhinein wie ein deutlich besserer Name wirkt 😄

Wir müssen aber glaube ich auch mal schauen, ob das überhaupt immernoch auftritt.

@Yagnap Yagnap self-assigned this Oct 9, 2024
@Yagnap Yagnap modified the milestones: next, 24.12 Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature a new feature or enhancement of existing functionality
Projects
None yet
Development

No branches or pull requests

3 participants