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

VIAs > easier way to use? #64

Open
sirjeannot opened this issue Sep 12, 2023 · 2 comments
Open

VIAs > easier way to use? #64

sirjeannot opened this issue Sep 12, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@sirjeannot
Copy link

Is your feature request related to a problem? Please describe.
The feature is not related to a problem, but to the ease of use.
The current implementation of the VIA makes use of nodes as an intermediate connection. It is much easier software-wise to implement so as you don't have much modifications to make to support this feature. However to create a VIA (or multiple VIAs) is quickly a pain on existing links as it involves removing the LINK and creating intermediates LINKS.
Having in the NODES and LINKS only the actual objects (and not the "sub" NODES and LINK in case of VIAs) would make managing the weathermaps much easier.
In the legacy Weathermaps, VIA is a parameter within a LINK. It is much easier this way to add a VIA once a Weathermap is created.

Describe the solution/feature you'd like
Being able to add VIA(s) inside the configuration of a LINK. Could be a node object within a LINK object?
Being able to specify a bending radius for a VIA

Additional context
I've created a script to migrate legacy Weathermaps/CactiDB to your plugin json format. So far I've avoided implementing VIAs migration. There are in some cases >100 objects on the graph and manually editing connection nodes becomes nightmarish.
Legacy WM, 2 VIA = 2 NODES, 1 Link, 2 VIA
Graf-WM, 2 VIA = 4 NODES, 3 LINKS

And again, thank you for the plugin. This is a much much needed one to move away from the legacy phpWeathermaps! 👍

@sirjeannot sirjeannot added the enhancement New feature or request label Sep 12, 2023
@nuchitpah
Copy link

Could you share the information you've edited regarding 'via'? I would like to do it

@sirjeannot
Copy link
Author

I'm not quite sure to understand the question. The edit to add a via consists in creating a 'c' node, changing the inital link a<>b to a<>c, then adding a new link c<>a (with all additional info such as width, query id, ...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants