Skip to content

fix: add missing --acc_precise arg and fix eval_one_file.sh bugs - #5

Open
jay-tau wants to merge 7 commits into
ornith-ai:mainfrom
jay-tau:fix/eval-script-and-docs
Open

fix: add missing --acc_precise arg and fix eval_one_file.sh bugs#5
jay-tau wants to merge 7 commits into
ornith-ai:mainfrom
jay-tau:fix/eval-script-and-docs

Conversation

@jay-tau

@jay-tau jay-tau commented Mar 8, 2026

Copy link
Copy Markdown

Closes #6
OOTB, the command does not run. Hence, the fix

Changes

eval_one_file.sh

  • Fix unknown option error message ($2$1)
  • Add required argument validation with a clear usage message
  • Fix rm to use -f flag and proper quoting
  • Remove redundant shift after arg parsing

README.md

  • Add missing --acc_precise to both example commands (offline and server)
  • Add --acc_precise to the arguments reference table
  • Change default --gpu_device_id from 7 to 0 in examples

jay-tau and others added 5 commits March 8, 2026 12:04
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jay-tau
jay-tau force-pushed the fix/eval-script-and-docs branch from 07cf83f to d4a561c Compare March 8, 2026 06:35
@jay-tau
jay-tau marked this pull request as ready for review March 8, 2026 06:47
@DeepReinforce
DeepReinforce force-pushed the main branch 4 times, most recently from 8ed993d to dbe0177 Compare March 30, 2026 06:44
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.

BUG: eval_one_file.sh fails due to missing --acc_precise and other bugs

1 participant