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

🧑‍💻added niaaml CLI #97

Merged
merged 1 commit into from
Jun 12, 2024
Merged

🧑‍💻added niaaml CLI #97

merged 1 commit into from
Jun 12, 2024

Conversation

LaurenzBeck
Copy link
Collaborator

‍💻added and documented niaaml CLI

💬 Description

This PR adds niaaml/cli.py and makes sure, that the CLI tool will be available when installing the package with pip.

❗ Issue Links

Fixes #84

🧪 How Has This Been Tested?

Nothing was changed in the rest of the package, which should qualify the PR as a minor change.
I made sure that the tests still pass locally though.

I tested the CLI locally with a test file:

niaaml optimize tests\tests_files\dataset_header_classes.csv --number-of-pipeline-evaluations 2 --number-of-inner-evaluations 2

afterwards, you can use the pipeline for inference:

niaaml infer tests\tests_files\dataset_header_classes.csv

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes

iaaml CLI
Copy link
Contributor

@sisco0 sisco0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I performed a review and changes are as expected.
No extra comments.

@firefly-cpp
Copy link
Owner

This is amazing. Thanks @LaurenzBeck and thanks also to @sisco0 for great ideas.

Supporting this CLI with man file may also be beneficial to the users.

@firefly-cpp firefly-cpp merged commit 47de5bc into master Jun 12, 2024
9 checks passed
@firefly-cpp firefly-cpp deleted the 84-cli branch June 12, 2024 11:19
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

Successfully merging this pull request may close these issues.

Provide CLI for easier interaction with NiaAML
3 participants