diagnose exit code

Offen
  • Stimmt etwas nicht?
#1,278 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
52/100
Issue-Typ
Feature
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
cpp
Bereich
cli

Rechercherichtung

Beginne in diagnose.cpp bei der return-Zeile und prüfe, wie die Prüfungen die Variable has_error setzen. Ändere das Verhalten des Exit-Status wie beschrieben und verifiziere anschließend, dass ein Diagnosefehler einen Status ungleich null erzeugt, während ein fehlerfreier Lauf erfolgreich bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Currently, the diagnose utility prints all output to stdout and returns 0 regardless of warnings. Would it make sense to return a non-zero exit status if there's an error with any of the checks?

This would make it easier to programmatically distinguish CmdStan runs with diagnostic warnings in the shell. As far as I know, currently the diagnose output needs to be checked manually or the output loaded into one of the programming language specific packages.

The return line in diagnose.cpp can just return the bool value from the variable has_error, or a more specific exit code can be used.

Vorherrschende Sprache
C++
Sterne
240
Forks
105
Ø Merge
1 T. 5 Std.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus stan-dev/cmdstan

Alle Issues in stan-dev/cmdstan

Ähnliche Issues

Weitere Issues zu C++

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.