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.

Sign up for GitHub

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

Jump to bottom

export inference models #7

Open
nhtlongcs opened this issue Sep 18, 2021 · 2 comments
Open

export inference models #7

nhtlongcs opened this issue Sep 18, 2021 · 2 comments
Assignees
@itskyf @nhtlongcs
Labels
enhancement New feature or request
Milestone
Sprint 4

Comments

@nhtlongcs
Copy link
Contributor

add export feature
@nhtlongcs nhtlongcs added the enhancement New feature or request label Sep 18, 2021
@nhtlongcs nhtlongcs added this to the Sprint 4 milestone Sep 18, 2021
@itskyf
Copy link
Contributor

itskyf commented Sep 23, 2021 •
edited
Loading

Right now it uses parse_argument inside the class, which makes users difficult to extent their arguments. I think we should modify opts class, so that it receives user's parser as parent, then use argparser's add_argument_group method

@nhtlongcs
Copy link
Contributor Author

Sure, would you be interested in sending a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@itskyf itskyf

@nhtlongcs nhtlongcs

Labels
enhancement New feature or request
Projects
None yet
Milestone
Sprint 4
Development

No branches or pull requests
2 participants
@itskyf @nhtlongcs