docs: required_columns description is incomplete for LLM and multimodal columns
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 65/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- python
- Ambito
- documentation
Direzione di ricerca
Inizia dall'elenco puntato required_columns in fern/versions/latest/pages/concepts/columns.mdx (intorno alle righe 222-230), poi leggi le proprietà required_columns in packages/data-designer-config/src/data_designer/config/column_configs.py per LLMTextColumnConfig, ImageColumnConfig e EmbeddingColumnConfig. È completato quando la documentazione elenca ogni sorgente: le variabili Jinja2 di prompt e system_prompt, i nomi delle colonne di multi_modal_context e target_column per gli embedding. Conferma il comportamento previsto con un maintainer prima di modificare, poiché la issue è in attesa di triage.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Priority Level
Low (Cosmetic / Minor)
Describe the bug
The concept documentation for required_columns in fern/versions/latest/pages/concepts/columns.mdx states that for LLM/Expression columns, dependencies are derived solely from Jinja2 {{ variables }}:
- For LLM/Expression columns: extracted from Jinja2 template
{{ variables }}
However, in packages/data-designer-config/src/data_designer/config/column_configs.py, LLMTextColumnConfig (and inheriting classes) also derives dependencies from:
- Jinja2 variables inside
system_prompt(when provided) - Column names referenced in
multi_modal_context(ctx.column_name)
Additionally, ImageColumnConfig derives dependencies from multi_modal_context, and EmbeddingColumnConfig depends on target_column.
The current documentation wording is incomplete regarding how DAG column dependencies are assembled. Updating this section will help users properly understand DAG generation ordering when using multimodal context and system prompts.
Steps/Code to reproduce bug
import data_designer.config as dd
col = dd.LLMTextColumnConfig(
name="summary",
model_alias="test-model",
prompt="Summarize this text",
system_prompt="Context: {{ user_persona }}",
multi_modal_context=[dd.ImageContext(column_name="image_input")]
)
# Inspect computed required_columns
print(col.required_columns)
# Output: ['user_persona', 'image_input']
# The docs state required_columns only extracts from Jinja2 template prompt,
# missing system_prompt and multi_modal_context dependencies.
Expected behavior
The required_columns section in columns.mdx should accurately list all sources of dependencies for LLM and multimodal columns:
- Variables in
promptandsystem_promptJinja2 templates ({{ variables }}) - Column names referenced in
multi_modal_context target_columnfor embedding columns
Agent Diagnostic / Prior Investigation
Investigated the documentation and source codebase:
- Checked
fern/versions/latest/pages/concepts/columns.mdx(lines 222–230). - Inspected implementation in
packages/data-designer-config/src/data_designer/config/column_configs.py:LLMTextColumnConfig.required_columns(lines 197–209): extracts fromself.prompt,self.system_prompt, andself.multi_modal_context.ImageColumnConfig.required_columns(lines 637–648): extracts fromself.promptandself.multi_modal_context.EmbeddingColumnConfig.required_columns(lines 589–591): returns[self.target_column].
- Confirmed that dataset builders rely on these exact dependencies for topological sorting in the DAG execution plan.
Additional context
I would be happy to contribute a fix after this issue is triaged and the intended behavior is confirmed.
Checklist
- I reproduced this issue or provided a minimal example
- I searched the docs/issues myself, or had my agent do so
- If I used an agent, I included its diagnostics above
- Lingua principale
- Python
- Stelle
- 2.3k
- Fork
- 227
- Merge medio
- 22h 51m
- PR unite (30g)
- 61
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un 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 NVIDIA-NeMo/DataDesigner
-
enforce \from future import annotations` via ruff FA102 rule`Forse già presa @chethanuk l’ha presa 37 giorni fa. Apertatask
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
NVIDIA-NeMo/DataDesigner#760 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
NVIDIA-NeMo/DataDesigner#1048 ·
I maintainer di solito rispondono entro 1 giorno
-
bug needs-discussion
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
NVIDIA-NeMo/DataDesigner#1045 ·
I maintainer di solito rispondono entro 1 giorno
-
Structured-output parsing drops valid dictionary entriesForse già presa @nabinchha l’ha presa 1 giorno fa. Apertabug
NVIDIA-NeMo/DataDesigner#1043 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
NVIDIA-NeMo/DataDesigner#1033 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di NVIDIA-NeMo/DataDesigner
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
infrastructure qa
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
khuisman/mcp-gee-sweet#1068 ·
I maintainer di solito rispondono entro 1 giorno
-
The book has a Nix install section that chelis.ch lacks, and it names two of the three systemsApertaarea:ecosystem documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Chelis-Lang/chelis#3866 ·
I maintainer di solito rispondono entro 1 giorno
-
`lvm_compact` has no self-hosted twin, and `docs/158` §5 says every language-face component mustApertabug sev-3 vm
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
issue: Picking a prompt from the / menu in a channel inserts a command chip instead of the promptApertabug confirmed issue
Difficoltà 2/5 Mezza giornata Idoneità per principianti 78/100
open-webui/open-webui#32712 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno