Skip to content
This repository was archived by the owner on Oct 19, 2024. It is now read-only.

CLI tool has wrong name for args #68

Open
moerlemans opened this issue Feb 22, 2024 · 0 comments · May be fixed by #79
Open

CLI tool has wrong name for args #68

moerlemans opened this issue Feb 22, 2024 · 0 comments · May be fixed by #79
Assignees
@moerlemans
Labels
bug Something isn't working

Comments

@moerlemans
Copy link
Contributor

On line 26 in ahcore/cli/data.py there is an error where manifest_fn = args.manifest_fn. However, this doesn't exist in the argparser. This should be args.manifest_uri. Also it is unclear in the argparser help that this uri should start with sqlite:///database_path
@moerlemans moerlemans added the bug Something isn't working label Feb 22, 2024
@moerlemans moerlemans self-assigned this Feb 22, 2024
@moerlemans moerlemans linked a pull request Apr 17, 2024 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@moerlemans moerlemans

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fixes arguments and also copies the folder for mrxs files NKI-AI/ahcore
1 participant
@moerlemans