lambdaisland / lambdaisland/deep-diff2
Diffing sets doesn’t deep-diff
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Clojure
- Sterne
- 326
- Forks
- 18
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
When diffing a set, i.e.
(ddiff/diff
#{{:db/id 17592186065894
:test "1"}}
#{{:db/id 17592186065894
:test "2"}
{:db/id 13194139559179
:db/txInstant #inst "2025-05-20T07:41:40.877-00:00"}})
the result is:
#{{:+
{:db/id 13194139559179,
:db/txInstant #inst "2025-05-20T07:41:40.877000000-00:00"}}
{:+ {:db/id 17592186065894, :test "2"}}
{:- {:db/id 17592186065894, :test "1"}}}
I would expect the result to be similar to a vector, i.e.
(ddiff/diff
[{:db/id 17592186065894
:test "1"}]
[{:db/id 17592186065894
:test "2"}
{:db/id 13194139559179
:db/txInstant #inst "2025-05-20T07:41:40.877-00:00"}])
produces:
[{:db/id 17592186065894, :test {:- "1", :+ "2"}}
{:+
{:db/id 13194139559179,
:db/txInstant #inst "2025-05-20T07:41:40.877000000-00:00"}}]
Digging deeper, it looks like there is bug with set diffing in general:
I.e.
(ddiff/diff #{{:id 123 :foo :bar}} #{{:id 123 :foo :baz}})
produces:
#{{:+ {:id 123, :foo :baz}} {:- {:id 123, :foo :bar}}}
where using a vector produces:
[{:id 123, :foo {:- :bar, :+ :baz}}]
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die ddiff/diff-Beispiele für Mengen und Vektoren nachzustellen, und vergleiche dann, wie verschachtelte Maps in jedem Fall abgeglichen werden. Als erledigt gilt die Aufgabe, wenn das Set-Diffing geänderte Felder innerhalb übereinstimmender Maps meldet und weiterhin tatsächlich hinzugefügte oder entfernte Elemente korrekt meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- clojure
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100