Marcel-Rodekamp / NSL

Nanosystem Simulation Library (NSL) implements statistical simulations for systems on the nanoscale
Other
2 stars 0 forks source link

Feature/command line #147

Closed Marcel-Rodekamp closed 1 year ago

Marcel-Rodekamp commented 1 year ago

Added a basic command line interface using CLIUtils. Further added a basic parameter class to store results of the command line parsing and for later use in other contexts.

GioPede commented 1 year ago

Maybe I have missed some of the latest meetings, but this whole PR does not seem to make a lot of sense to me... I understand having some way to manage the inputs needed, but I wouldn't add an external dependency nor allow arbitrary parameters to be set from CLI if we go the inputfile route: it is way more efficient, extensible, type-safe and also forces you to keep track of the runs you have made in an explicit way. I'm open to discussing this more if you want

Marcel-Rodekamp commented 1 year ago

Maybe I have missed some of the latest meetings, but this whole PR does not seem to make a lot of sense to me... I understand having some way to manage the inputs needed, but I wouldn't add an external dependency nor allow arbitrary parameters to be set from CLI if we go the input file route: it is way more efficient, extensible, type-safe and also forces you to keep track of the runs you have made in an explicit way. I'm open to discussing this more if you want

Are the changes in 81cfb567db0f4594dd1498f9380f71d2c47e7219 sufficient to address the results of todays discussion.

GioPede commented 1 year ago

This all looks fine now. We still might need to revise the strategy in the future, but it is a big improvement