Skip to content

Fixes issue in results path#87

Merged
lmdiazangulo merged 4 commits into
devfrom
fixes-issue-with-results-paths
May 15, 2026
Merged

Fixes issue in results path#87
lmdiazangulo merged 4 commits into
devfrom
fixes-issue-with-results-paths

Conversation

@lmdiazangulo
Copy link
Copy Markdown
Collaborator

No description provided.

@adrianarce-elemwave
Copy link
Copy Markdown
Collaborator

LauncherTest.nested_shield_resistance_and_transfer_impedance_written_to_output_json failed with following message:

unknown file: Failure
C++ exception with description "[json.exception.parse_error.101] parse error at line 1, column 1: attempting to parse an empty input; check that your input string or stream contains the expected JSON" thrown in the test body.

@lmdiazangulo
Copy link
Copy Markdown
Collaborator Author

@copilot check what's happening

Copilot AI and others added 3 commits May 14, 2026 10:03
Agent-Logs-Url: https://github.com/OpenSEMBA/tulip/sessions/bd41a440-d3a7-46c7-b361-ad0a6ee03f8c

Co-authored-by: lmdiazangulo <4919398+lmdiazangulo@users.noreply.github.com>
Agent-Logs-Url: https://github.com/OpenSEMBA/tulip/sessions/bd41a440-d3a7-46c7-b361-ad0a6ee03f8c

Co-authored-by: lmdiazangulo <4919398+lmdiazangulo@users.noreply.github.com>
Agent-Logs-Url: https://github.com/OpenSEMBA/tulip/sessions/bd41a440-d3a7-46c7-b361-ad0a6ee03f8c

Co-authored-by: lmdiazangulo <4919398+lmdiazangulo@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026

LauncherTest.nested_shield_resistance_and_transfer_impedance_written_to_output_json failed with following message:

...

Fixed in commit 7263f04. The launcher now writes results back to <caseName>.tulip.out.json under the selected output directory, so LauncherTest.nested_shield_resistance_and_transfer_impedance_written_to_output_json no longer reads an empty/missing JSON file.

@copilot check what's happening

I traced it to commit 5831ac8 removing the case-name file prefix from the launcher output path, which made tests look for <case>.tulip.out.json while the code wrote tulip.out.json. I restored the expected output-path behavior in commit 7263f04.

@lmdiazangulo lmdiazangulo merged commit 7378c26 into dev May 15, 2026
2 checks passed
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.

3 participants