Feature request: extend `merge_update` into BoxLists
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia individuando il comportamento di merge_update e la gestione di BoxLists descritti nell'issue, quindi esamina i test esistenti relativi a merge, se disponibili. Confronta la sostituzione attuale dei dati con l'aggiornamento previsto elemento per elemento e definisci il completamento come la conservazione dei dizionari di liste esistenti aggiungendo al contempo i campi degli elementi in ingresso corrispondenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Current
If I have these two BoxDicts:
In [1]: from box import Box
...: dict1 = Box({'data': [{'foo': 1, 'foobar': 20}, {'bar': 2}] }, box_dots=True)
...: dict2 = Box({'data': [{'foo': 1, 'baz': 10}] }, box_dots=True)
...: print(dict1.keys(dotted=True))
...: print(dict2.keys(dotted=True))
['data[0].foo', 'data[0].foobar', 'data[1].bar']
['data[0].baz', 'data[0].foo']
Then, if I use merge_update I get:
In [2]: dict1 + dict2
Out[2]: Box({'data': [{'foo': 1, 'baz': 10}]})
Expected behaviour
When merging the two datasets, it would be great if Box entered each list element and updated the dictionary:
In [1]: dict1 + dict2
Out[1]: Box({'data': [{'foo': 1, 'foobar': 20, 'baz': 10}, {'bar': 2}]})
Since the BoxDict has a pseudo-key for elements of the lists (e.g., 'data[0].foo') it feels like it should be possible.
Versions
python: 3.11
box: 7.1.1
- Lingua principale
- Python
- Stelle
- 2.8k
- Fork
- 140
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
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 cdgriffith/Box
-
subclass of `Box` does not unpickle correctlyForse già presa @koriyoshi2041 l’ha presa 135 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
cdgriffith/Box#308 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
cdgriffith/Box#307 · 1 commento ·
-
Dotted keys read from YAML are lost with box_dots / default_boxForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
cdgriffith/Box#305 ·
-
[Bug] Camel killer box and box dots gives unexpected behaviourForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 30/100
cdgriffith/Box#300 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
cdgriffith/Box#298 ·
Tutte le issue di cdgriffith/Box
Issue simili
-
needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
darnitdevorg/darnit#587 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
I maintainer di solito rispondono entro 1 giorno
-
accessibility good first issue
Difficoltà 1/5 1-3 ore Idoneità per principianti 92/100
I maintainer di solito rispondono entro 1 giorno
-
llm translation
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
bug bug-hunt
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno