Repository navigation
Add real transformer batch inference - #116
Mayukhga83 wants to merge 1 commit into
Conversation
|
Thanks for the work here, @Mayukhga83. The batched path is careful, and it matched one-by-one output on the real ModernBERT model at every batch size I tried. #113 implemented the same change for #23, was opened first, and has now been merged; our policy for duplicate PRs is that the earlier one lands when it is correct, so I'm closing this one. For context on one difference: with a tokenizer set to A few pieces here did not land with #113 and would be welcome as small, separate PRs on top of
Thanks again, and we'd be glad to review any of these. |
Summary
resourcemoduleRelated issue
Closes #23.
Type of change
Testing
ruff format --check lettucedetect/ lettucedetect_api/ tests/ruff check lettucedetect/ lettucedetect_api/ tests/ --extend-exclude lettucedetect/integrations/python -m pytestruff check scripts/benchmark_detectors.pyChecklist
Rights & sign-off (required)
distributed under the repository's MIT license
(see CONTRIBUTING).