Carlo Bernaschina
Carlo Bernaschina
Extra info: To reproduce it is enough to build a model with `100000` nodes connected in a line `n0 => n1 => n2 => .... => n100000`.
Thank you very much for these fixes. Is there an idea of when these changes will get into a release?
Just to fully understand. The dev release you are suggesting would be for cherry picking the changes into `elkjs` and have a point release there?
That would be awesome
Friendly Ping. Will you have time to `assemble a dev release based on the current master for elkjs`?
Version 0.7.3-dev solved the problem with `$connectedComponentsDFS`, but it is triggering it again in `$tightTreeDFS`. https://github.com/eclipse/elk/blob/3630e23fa5abc253233296e2bdf5e18828e5dba7/plugins/org.eclipse.elk.alg.common/src/org/eclipse/elk/alg/common/networksimplex/NetworkSimplex.java#L515-L537 Which is also recursive
I tried to change the strategy to any other value, e.g.: ```javascript layoutOptions: { // ... 'elk.layered.nodePlacement.strategy': 'LINEAR_SEGMENTS', } ``` But nothing changes. (let me know if I'm doing it...
In the end I added a regular expression that fixes this and other problems.  I am not sure here. It's necessary to use no-break spaces otherwise IE would collapse...
And Chrome is sending a warning about the method used for the animation. > SVG's SMIL animations (, , etc.) are deprecated and will be removed. Please use CSS animations...
Note that a lot of this code could be simplified by using `std::min` and `std::max`. `std::clamp` could also be used in various places, but that would be limited to C++17...