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

unclear failure in case of empty suffix #318

Open
rusefillc opened this issue Aug 2, 2023 · 2 comments
Open

unclear failure in case of empty suffix #318

rusefillc opened this issue Aug 2, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@rusefillc
Copy link
Collaborator

https://github.com/rusefi/stim/actions/runs/5734374341/job/15540458399

* Checking Python module 'NumPy': OK
* Checking Python module 'cairocffi': OK
* Checking module dependencies: ffi and cairo
* Checking ffi...
* Library ffi FOUND!
* Checking cairo...
* Library cairo FOUND!
* Checking Python module 'gzip': OK
* Checking Python module 'simpleparse': OK
* Checking Python module 'PyVRML97': OK
All checks done!
Processing digital_inputs board...
_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.
Removing old files of the board...
Creating digital_inputsa-bom_replace_digital_inputs-a.csv...
* appending the CPL with offset (0,0)...
Traceback (most recent call last):
  File "bin/process_board.py", line 264, in <module>
    append_cpl(frame_path + "/" + frame_name + "-CPL.csv", board_cpl, "0", "0", "0", 0)
  File "bin/process_board.py", line 113, in append_cpl
    with open(src_fname, 'rt') as src_f, open(dst_fname, 'a') as dst_f:
FileNotFoundError: [Errno 2] No such file or directory: '/digital_inputs/boards/digital_inputsa-bom_replace_digital_inputs-a.csv/frame/a-CPL.csv'
ABORTING!

what is this about? can we have better failure message to help users like myself?

@andreika-git
Copy link
Owner

That's about empty board suffix, which is passed as a command line argument, so the whole command line is parsed incorrectly (arguments shifted)

@rusefillc
Copy link
Collaborator Author

@andreika-git thank you! let me work around it in https://github.com/rusefi/stim but we need to improve usability

@rusefillc rusefillc changed the title unclear failure unclear failure in case of empty suffix Aug 2, 2023
@rusefillc rusefillc added the enhancement New feature or request label Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants