Skip to content

Bugfix: append traceback to existing output for errored runs#478

Open
LudovicoRighi wants to merge 3 commits intomainfrom
output-errored-runs
Open

Bugfix: append traceback to existing output for errored runs#478
LudovicoRighi wants to merge 3 commits intomainfrom
output-errored-runs

Conversation

@LudovicoRighi
Copy link
Copy Markdown
Collaborator

@LudovicoRighi LudovicoRighi commented Jul 11, 2025

Related note:

apsis/python/apsis/apsis.py

Lines 339 to 340 in e5f1029

# FIXME: For now, use the name "output" as this is the only one
# the UIs render. In the future, change to "traceback".


Tested this for:

runs that produce output

output261 bytes
some_output...
some_output...
some_output...
Traceback (most recent call last):
  File "/home/lrighi/apsis/python/apsis/running.py", line 101, in _process_updates
    raise FileNotFoundError("TTTSimulating an issue!")
FileNotFoundError: TTTSimulating an issue!

runs that don't produce any output

output216 bytes
Traceback (most recent call last):
  File "/home/lrighi/apsis/python/apsis/running.py", line 101, in _process_updates
    raise FileNotFoundError("TTTSimulating an issue!")
FileNotFoundError: TTTSimulating an issue!

UPDATE: need to review this again; we probably shouldn't mix output and Apsis errors.

@LudovicoRighi LudovicoRighi requested a review from gusostow July 11, 2025 16:42
@LudovicoRighi LudovicoRighi changed the base branch from master to main July 16, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants