Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Let a use_cfgs entry pin its own tool

Aperta
  • Qualcosa non va?
#253 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
68/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
build-system

Direzione di ricerca

Inizia tracciando create_instance e _load_child_cfg in flow/base.py attraverso la closure della factory in flow/factory.py, quindi esamina _CMDLINE_FIELDS in flow/hjson.py. Aggiungi la gestione dello strumento per ciascun child, preservando la precedenza della riga di comando e rifiutando le altre chiavi di override, quindi aggiungi un test di regressione del caricamento che mostri che child xcelium e vcformal separati usano le proprie impostazioni dello strumento e i propri percorsi scratch.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement

Part of lowRISC/dvplan#129.

Running a simulation regression under Xcelium and a formal regression under VC Formal in one invocation is a requirement for the bundle work, and it cannot be expressed today.

tool is the one key a config cannot set for itself once the command line gives it, by way of _CMDLINE_FIELDS in flow/hjson.py. Scalar merging is first-non-default-wins and raises on a genuine conflict, so a config setting tool: vcformal while importing OpenTitan's common_formal_cfg.hjson, which pins tool: jaspergold, fails to load with a conflicting-value error. The only way to select VC Formal is --tool vcformal on the command line, which applies to every config in the invocation.

Scope

  • Let a use_cfgs entry carry a tool, seeded into that child's initial_values. Because arg_keys is computed from the keys present in initial_values, a seeded value wins over every imported file, which is exactly the mechanism --tool already uses. No changes needed in any OpenTitan or Mocha config.
  • create_instance and _load_child_cfg in flow/base.py take the per-child overrides, and the factory closure in flow/factory.py applies them.
  • --tool on the command line keeps overriding everything, since that is what it is for.
  • Reject an override for a key other than tool for now, so this does not quietly become a general-purpose per-child config channel. That belongs with #36.

Done when

  • A primary config with one child pinned to xcelium and another pinned to vcformal loads, and each child's scratch path and tool config come from its own tool.
  • A test covers the load, since the failure mode today is a RuntimeError from config merging and is easy to reintroduce.
Lingua principale
Python
Stelle
15
Fork
22
Merge medio
4g 46m
PR unite (30g)
6

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di lowRISC/dvsim

Tutte le issue di lowRISC/dvsim

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.