← 输入框|github.com|刷新
Skip to content

chore: remap chipper to use chipperv2 - #300

Merged
awalker4 merged 1 commit into
mainfrom
austin/chipper-mapping
Nov 3, 2023
Merged

awalker4 merged 1 commit into
mainfrom
austin/chipper-mapping

Conversation

@awalker4

@awalker4 awalker4 commented Nov 3, 2023 •

Copy link
Copy Markdown
Collaborator

We want this to alias to the current model. Also, wrap the error that comes from chipperv1 becoming a private model.

To verify:

  • Run the server
make run-web-app
  • A call to chipperv1 should give a useful error:
curl -X POST 'http://localhost:8000/general/v0/general' --form files="@$file" --form strategy=hi_res --form hi_res_model_name=chipperv1
{"detail":"The Chipper model is not available for download. It can be accessed via the official hosted API."}
  • Change the model name to chipper. You should see chipperv2 in the log message containing partition input data

We want this to alias to the current model. Also, wrap the error that comes from chipperv1 becoming a private model.
Comment thread CHANGELOG.md Resolved

@yuming-long yuming-long left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • 'hi_res_model_name=chipper' give me error: "chipper model is not available for download. It can be accessed via the official hosted API." (error from chipper v2)
  • 'hi_res_model_name=chipperv1' gives me the json output (chipper v1 should still work)

@yuming-long yuming-long left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested this again after setting the HF token (so previous comment is from without token set)

  • output from chipper v2: the first element has text: LayoutParser: A Unified Toolkit for Deep Learning Based Document Image Analysis
  • output from chipper v1: the first element has text: arXiv: 2103.15348v2 [cs.CW] 21 Jun 2021
  • output from chipper: the first element has text: LayoutParser: A Unified Toolkit for Deep Learning Based Document Image Analysis, same as chipperv2

@awalker4
awalker4 merged commit 5e04b1c into main Nov 3, 2023
@awalker4
awalker4 deleted the austin/chipper-mapping branch November 3, 2023 20:31
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.

2 participants