search

FAIRmat-NFDI / pynxtools

https://fairmat-nfdi.github.io/pynxtools/
Apache License 2.0
13 stars 8 forks source link

Suggestion for fixing the unintentional situation that since v0.6 the… #404

Closed mkuehbach closed 3 months ago

mkuehbach commented 3 months ago

… dataconverter required a config file if it was called the classical way and in case no parameter yaml file was provided. For programmatically use of the converter the creation of an additional params file with only a few entries is unnecessary overhead especially when all cli arguments are available in the callers scope and thus the CLI call can be issues directly using the classical way dataconverter convert [INPUTFILES] options

mkuehbach commented 3 months ago

I also removed the outdated documentation of the command line options as it is a second place where changes have to be applied constantly and the instructions for empowering people to find these are already given in the readme plus follow the typical style how to get help for command line calls

mkuehbach commented 3 months ago

Tested 30ad68c of this branch solves the problems @sherjeelshabih