Query-algebra spec: restriction by an expression with no common attributes, and restricting dj.U() aggregates
I maintainer di solito rispondono entro 3 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 55/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Ambito
- databases, documentation
Direzione di ricerca
Start with make_condition in condition.py to confirm the common-attribute matching and the no-shared-attributes rule, then edit reference/specs/query-algebra.md at §2.4 (line 103), §2.5 (line 124) and §8.3 (line 956). Also add the universal-aggregate example to reference/operators.md around line 258. Done means the matching rule is stated on common attributes with a link to §9, the empty-attribute rule has examples, and the universal-aggregate restriction is documented with the .make_subquery() workaround until datajoint-python#1580 is resolved.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
reference/specs/query-algebra.md leaves out two behaviors that the implementation defines, and misstates the matching rule for restriction by a query expression.
1. §2.5 states the matching rule as primary keys
Line 124: "Restrict to rows with matching primary keys in another expression." The implementation (condition.py, make_condition) matches on every namesake attribute the two expressions share, subject to the lineage check of semantic matching, not only on primary-key attributes.
Fix: state the rule as matching on common (homologous) attributes, and link §9 (Semantic Matching).
2. §2.5 omits restriction when the expressions share no attributes
§2.4 documents the dictionary case (line 103: "If NO keys match, condition evaluates to True"). The expression case is undocumented. The implementation treats it the same way (condition.py: "without common attributes, any non-empty set matches everything"):
A & Bwith no common attributes: all ofAifBis non-empty, none ifBis empty.A - Bwith no common attributes: none ofAifBis non-empty, all ifBis empty.
This is what makes a global gate possible, for example Table & (dj.U().aggr(Other, n="count(*)") & "n > 0").
Fix: add the rule to §2.5 with an example, next to the dictionary rule in §2.4.
3. §8.3 does not cover restricting a universal aggregate
§8.3 shows dj.U().aggr(Student, n='count(*)') returning one row, and line 956 says restrictions after aggregation go to HAVING or a subquery, but neither covers restricting an ungrouped aggregate (dj.U().aggr(T, n="count(*)") & "n = 3"). In v2.3.4 that raises an AssertionError (datajoint/datajoint-python#1580), and len() of the subquery workaround raises a SQL syntax error (datajoint/datajoint-python#1581).
Fix: once #1580 is resolved, add a §8.3 example that restricts a universal aggregate and uses it as a restriction (rule 2 above). Until then, note the .make_subquery() workaround. The "Universal Aggregation (Single Row Result)" section of reference/operators.md (line 258) could take the same example.
- Lingua principale
- Jupyter Notebook
- Stelle
- 7
- Fork
- 12
- Merge medio
- 2g 19h
- PR unite (30g)
- 20
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna 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 datajoint/datajoint-docs
-
documentation
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
datajoint/datajoint-docs#301 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 3/5 Mezza giornata Idoneità per principianti 52/100
datajoint/datajoint-docs#299 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
datajoint/datajoint-docs#298 ·
I maintainer di solito rispondono entro 3 giorni
-
documentation
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
datajoint/datajoint-docs#292 ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di datajoint/datajoint-docs
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
datajoint/datajoint-python#1586 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue hacktoberfest
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
libredb/libredb-studio#1755 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(accounts): Balance Sheet and Profit and Loss crash on PostgreSQL due to MySQL force_indexAperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Bible-Research/bible_research#51 ·
I maintainer di solito rispondono entro 6 giorni