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
which may have some benefits, but I think it has a huge issues: objects are not sorted so if you want to return annotations in order for just a range of characters (which I suspect is a fairly common use case), you have to read all the annotations, sort them by spans, and then filter the results by number of characters. That's a huge performance issue, I suspect this might impact the UI of the editor as well
The text was updated successfully, but these errors were encountered:
@eroux right, but for annotation lookup operation with annotation id, then we need to linear search over all the annotations. Anyway this is much less expensive than sorting and filtering.
sure, I will refactor to store annotations into list with order of span.
currently inside a layer annotations are stored in an object:
which may have some benefits, but I think it has a huge issues: objects are not sorted so if you want to return annotations in order for just a range of characters (which I suspect is a fairly common use case), you have to read all the annotations, sort them by spans, and then filter the results by number of characters. That's a huge performance issue, I suspect this might impact the UI of the editor as well
The text was updated successfully, but these errors were encountered: