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

[CLI] Check the required parameters #9

Closed
delhomer opened this issue Jan 10, 2024 · 2 comments
Closed

[CLI] Check the required parameters #9

delhomer opened this issue Jan 10, 2024 · 2 comments
Assignees

Comments

@delhomer
Copy link
Owner

The CLI defines kpconv preprocess, kpconv train, kpconv test, kpconv visualize and kpconv plotconv. These commands share parameters, some of them are required for a subset of commands. One should doublecheck the parameter list and if they should be required for each command.

@AuBousquet
Copy link
Collaborator

AuBousquet commented Jan 19, 2024

kpconv visualize, kpconv plotconv have been given up in this branch. They are not used in the context of this project. The original .py files are kept in the sources, but they are not available from the command line interface anymore.

The documentation has been updated, to present the CLI options, for each command.

With the current version of the python code, it is not possible to avoid having the two options -f and -d for the kpconv test command, since the program calls the -d datapath at two lines, here and there. Apparently, the data used to train the model are still necessary for the test step.

@AuBousquet
Copy link
Collaborator

#22

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

No branches or pull requests

2 participants