Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass "args" via .json file rather than arguments to rust program #20

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

drojf
Copy link
Contributor

@drojf drojf commented May 16, 2024

I wanted an easier way to pass data between the python and rust program (instead of the command line), so I made the python script write to a temporary .json file which the Rust file reads.

I guess it could just pass it via stdin instead, but I was just quickly writing something up so I just did whatever worked.

However, the thing I needed the extra settings for turned out not to be required.

But if in the future more things need to be passed between the scripts, this PR can be used. Just need to :

  • Remove the "path ID override" function, I don't think we will end up using it in the future
  • Pass JSON via stdin? unsure how other command line programs normally do it. I guess the temp file method works fine, though.

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