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

Add support for model tag selection in cli --model #59

Closed
abhimskywalker opened this issue Sep 13, 2023 · 1 comment · Fixed by #60
Closed

Add support for model tag selection in cli --model #59

abhimskywalker opened this issue Sep 13, 2023 · 1 comment · Fixed by #60

Comments

@abhimskywalker
Copy link
Contributor

The default model tag is 0.1.0-small. If we want to use the base model (or other tags in future), we should be able to use 0.1.0-base as a model tag liek this:

$ nougat path/to/file.pdf -o output_directory -m 0.1.0-base
@abhimskywalker
Copy link
Contributor Author

I tried parsing a few PDFs with small version but it gave MISSING_PAGE on some of the pages, while the base model was able to convert the pages perfectly.

So have raised #60 as a PR towards this support for model selection at cli level to help both myself and others. :)

I could have perhaps just opened the PR and not an issue independently, but wanted to ensure this issue captures any other user's concerns around this if they have similar concerns. Apologies in advance if this increased any workload.
Have linked the issue to PR so this should close along with the PR whenever it's merged.

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 a pull request may close this issue.

1 participant