memgraph / orb

Graph visualization library
Apache License 2.0
364 stars 17 forks source link

New: Refactor simulator #38

Closed cizl closed 1 year ago

cizl commented 1 year ago

Rework simulator logic to better handle physics changes and data updates. Fixes https://github.com/memgraph/orb/issues/18

This can be reviewed now. There is still some polishing to be done but I need feedback on the current status and some resolutions to the questions that are commented in the code.

I also have some questions on the expected behavior of some scenarios and what we can add as reasonable defaults in the settings.

This is a significant change from the API side, now .render() doesn't simulate. Rather orb.data.merge() and other orb.data callbacks issue a simulation event. @tonilastre Feel free to browse the code a bit but maybe a call would be more efficient to resolve all questions and get you up to speed.