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

Configuration File Generator #27

Open
wants to merge 70 commits into
base: main
Choose a base branch
from
Open

Conversation

mmouchous-ledger
Copy link
Member

@mmouchous-ledger mmouchous-ledger commented Jan 17, 2025

This PR intends to introduce a Configuration file generator for LaserStudio, when it is not found at startup.

  • Laserstudio search for a 'config.yaml' file from current directory up to root directory.
  • If it do not find it, a wizard is created and by parsing the schema in the project it creates a Configuration file, which can be saved and used by Laser Studio.

Documentation site is also updated in this PR (https://laserstudio.readthedocs.io/en/schema_generator/)

@mmouchous-ledger mmouchous-ledger changed the title [WIP] Configuration File Generator Configuration File Generator Jan 21, 2025
@mmouchous-ledger mmouchous-ledger marked this pull request as ready for review January 21, 2025 09:08
@kingofpayne
Copy link
Contributor

Regarding the NIT camera configuration, the configuration wizard shows path to the NUC files. It could be nice to have a "browse" ("...") button on the right to select the files with a open file dialog.

@kingofpayne
Copy link
Contributor

Generated devices path in the resulting configuration file are wrong when selecting the device using the drop down menu. For instance, it generated this in the configuration file

stage:
  dev: '/dev/ttyUSB0 [sn: REDACTED | info: USB VID:PID=0403:6001 SER=REDACTED LOCATION=1-1.1]'

The software cannot load this path. I believe it should generate this instead:

stage:
  dev: '/dev/ttyUSB0'

@mmouchous-ledger
Copy link
Member Author

Regarding the NIT camera configuration, the configuration wizard shows path to the NUC files. It could be nice to have a "browse" ("...") button on the right to select the files with a open file dialog.

This may have be due to the edition of the field, entering an entry slightly different (insertion of space removal of one character) which may have
"/dev/ttyUSB0 [sn: REDACTED | info: USB VID:PID=0403:6001 SER=REDACTED LOCATION=1-1.1] " (for instance) actually present in the field.

To reduce this to happen, when the user selects one line, the editable field will contain only "/dev/ttyUSB0"

Note: The field is editable to permit the user to give a dev path which may not exist at the generation of the configuration file (ie the device is not yet connected)

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.

config.yaml file generator
2 participants