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
adapts the mapping locally (the rendered data is updated directly)
ensures a saved state (so that later mapping-requests provide correct values)
unblocks the user
Multiple ideas to improve this:
Idea 1: Don't ensure a saved state
when merging two ids A and B (e.g., so that B is changed to A) one can adapt the results of mapping-requests (that the server sent without knowing about the merge) by reassigning B to A
ensuring a saved state would not be necessary anymore then
however, split operations would need a new synchronization point (before and after the split; previously it was only after the split). the before-sync is only needed when a merge wasn't saved yet.
Idea 2: Don't block the user while ensuring a saved state
the user should be free to move around and inspect stuff. nothing bad should happen when only a save is pending. they don't need to know that wk is waiting for a save.
if the user does a split-operation after a merge, the merge needs to be awaited before the split operation is sent to the server, though. thus, we would defer the synchronization point.
Both ideas introduce additional complexity. I'd prefer doing (1) for now, since I have a bad feeling about dissolving the busy-blocking.
The text was updated successfully, but these errors were encountered:
philippotto
changed the title
Improve merge speed in proofreading tool by reducing roundtrips
[Proofreading] Improve merge speed by reducing roundtrips
Aug 1, 2024
philippotto
changed the title
[Proofreading] Improve merge speed by reducing roundtrips
[Proofreading] Improve merge speed by not awaiting roundtrips
Aug 1, 2024
no, unfortunately not. this issue was created to suggest further improvements for the "magic mappings" performance. probably not high-pri for now, but it could definitely make the merge operation more snappy (think old merger-mode style).
Currently, a merge operation:
Multiple ideas to improve this:
Idea 1: Don't ensure a saved state
Idea 2: Don't block the user while ensuring a saved state
Both ideas introduce additional complexity. I'd prefer doing (1) for now, since I have a bad feeling about dissolving the busy-blocking.
The text was updated successfully, but these errors were encountered: