Properly separate ydoc-sync from custom comm messages
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
Direzione di ricerca
Inizia da src/notebookrenderer/yCommProvider.ts, in _onMsg, e segui come vengono distribuiti i messaggi con buffer binari. Esamina la gestione dei comm correlata in ypywidgets prima di definire un protocollo che distingua i messaggi di sincronizzazione ydoc dai messaggi personalizzati. Il lavoro è completato quando i messaggi comm personalizzati contenenti buffer binari non vengono più elaborati come messaggi di sincronizzazione ydoc e la breaking change è coordinata tra entrambi i progetti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In yCommProvider, _onMsg treats any message with binary buffers as a ydoc sync message and everything else as a custom message. This is unfortunate, as we wouldn't be able to send custom comm messages that contain binary buffers — they'd be trapped here and processed as ydoc sync messages.
For now this is fine, since we're only partially supporting custom comms. But we should consider a breaking change in both ypywidgets and yjs-widgets to properly define a comm protocol with ydoc sync messages properly separated, e.g. branching on msg.type === 'ydoc-sync' instead of the buffers check.
Originally raised by @martinRenou in https://github.com/QuantStack/yjs-widgets/pull/30#discussion_r3587111607.
- Lingua principale
- TypeScript
- Stelle
- 10
- Fork
- 10
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di QuantStack/yjs-widgets
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
QuantStack/yjs-widgets#8 · 5 commenti ·
Tutte le issue di QuantStack/yjs-widgets
Issue simili
-
ADD openalgoApertatemplate
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
factory-active factory-automatic task-bug-reproduction-success task-identify-harness-labels-done task-identify-issue-type-done
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
vercel/ai#21528 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
bug Needs: Triage :mag:
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
microsoft/fluentui-contrib#671 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
sveltejs/acorn-typescript#150 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100