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

Replace generated code by runtime CSV parsing #1

Open
bikeshedder opened this issue Feb 8, 2025 · 0 comments
Open

Replace generated code by runtime CSV parsing #1

bikeshedder opened this issue Feb 8, 2025 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bikeshedder
Copy link
Owner

It sounded like a great idea in the beginning but the actual benefit is rather neglectable. Anyone using this software is going to use only the APIs anyways. It isn't beneficial at all to have the parameters generated as constants.

I want to embed the params.csv, enums.csv and devices.csv (does not exist) in the binary similar to the default config and provide commands to export them. This would also enable switching between parameter definitions in order to support older ROTEX HPSUs.

This should even make the implementation a bit simpler as well.

@bikeshedder bikeshedder added the enhancement New feature or request label Feb 8, 2025
@bikeshedder bikeshedder self-assigned this Feb 8, 2025
@bikeshedder bikeshedder added this to the 0.2 milestone Feb 8, 2025
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

1 participant