Skip to content

fix: create output directory instead of its parent in make_onnx.py - #71

Draft
andrewwhitecdw wants to merge 1 commit into
NVlabs:masterfrom
andrewwhitecdw:bugfix/make-onnx-ea3b8f4e
Draft

andrewwhitecdw wants to merge 1 commit into
NVlabs:masterfrom
andrewwhitecdw:bugfix/make-onnx-ea3b8f4e

Conversation

@andrewwhitecdw

@andrewwhitecdw andrewwhitecdw commented Jul 27, 2026

Copy link
Copy Markdown

Problem

fix: create output directory instead of its parent in make_onnx.py

Fix

Replace:

    os.makedirs(os.path.dirname(args.save_path), exist_ok=True)

with:

    os.makedirs(args.save_path, exist_ok=True)

Files changed

  • scripts/make_onnx.py

Reconstructed from original sweep finding patch(es).
@andrewwhitecdw
andrewwhitecdw force-pushed the bugfix/make-onnx-ea3b8f4e branch from c392bc6 to 6c2e5b4 Compare August 2, 2026 19:36
@andrewwhitecdw

Copy link
Copy Markdown
Author

Closing after multiple repair attempts: original patch no longer applies or auditor feedback remains unaddressed.

@andrewwhitecdw
andrewwhitecdw deleted the bugfix/make-onnx-ea3b8f4e branch August 2, 2026 23:56
@andrewwhitecdw
andrewwhitecdw restored the bugfix/make-onnx-ea3b8f4e branch August 17, 2026 20:33
@andrewwhitecdw

Copy link
Copy Markdown
Author

script closed on accident

This branch has not been deployed

No deployments
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.

1 participant