You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's currently a lot of areas of the code where callbacks and objects could be memoized but are not. Some help here would be appreciated.
Only thing to note is the fact that nodes and edges (which are both objects) are at the core of everything makes things a little difficult I believe — as we'd need to do a deepequals check in some places which might end up being even more expensive?
The text was updated successfully, but these errors were encountered:
ideally we have some evidence that these changes actually improve performance and aren't just adding more complexity, as useCallback/useMemo do add some overhead if used incorrectly
There's currently a lot of areas of the code where callbacks and objects could be memoized but are not. Some help here would be appreciated.
Only thing to note is the fact that nodes and edges (which are both objects) are at the core of everything makes things a little difficult I believe — as we'd need to do a deepequals check in some places which might end up being even more expensive?
The text was updated successfully, but these errors were encountered: