[Other] setup.sh --update --build does not compile the engine (the engine step is skipped entirely)

Aperta Adatta ai principianti
  • Qualcosa non va?
#5,214 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 4 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
bash, cpp, python
Ambito
build-system, cli

Direzione di ricerca

Inizia da setup.py, intorno a update_install alla riga 7250, e dal controllo plain_start; confronta i percorsi --update --build e --build con il testo della guida per --build. Segui il punto in cui viene chiamato build_engine_hip e controlla i test pertinenti dello script di configurazione o i comandi di compilazione documentati. Il lavoro è completato quando il comando di aggiornamento compila il motore dai sorgenti e --build da solo non avvia il modello.

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

Descrizione

Area

[Other] (anything else, including setup and installation)

Desktop app, Terminal or source?

Source (START-HERE.bat, setup.sh or a build)

What happened

On a source checkout, ./setup.sh --update --build finished in about one second with Strata is updated (engine 0.1.55) and did not compile anything. The source tree was 0.1.56 and engine/BUILD.json still said 0.1.55. Deleting engine/BUILD.json first changes nothing.

--build alone is also not in the plain_start guard, so ./setup.sh --build starts the model instead of compiling.

Cause (static analysis). setup.py:7250 in update_install:

if not a.build:
    update_installed_engine(a.prebuilt)

With --build, neither the ready-made fetch nor build_engine_hip runs. The help text for --build says "compile the engine instead of using the ready-made one", so the combination should compile.

Smallest steps or command that reproduce it
  1. rm -f engine/BUILD.json
  2. ./setup.sh --update --build
  3. cat engine/BUILD.json or check the engine version: unchanged.

Expected: the engine is compiled from the current source.
Actual: no compile, and the line says "Strata is updated".

Proposed fix: route a.build to the compile path, or at minimum warn that --update --build does not compile and point at ./setup.sh --setup --build.

Reproduction status

Reproduced locally

Strata version

Strata 0.2.10 source, engine 0.1.55 at the time

Graphics card, memory and system

Radeon AI PRO R9700 32 GB, 121.5 GiB RAM, Fedora 44 (kernel 7.2.13-200.fc44.x86_64)

Lingua principale
C++
Stelle
11.6k
Fork
1k
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 Niko1221/Strata

Tutte le issue di Niko1221/Strata

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.