Skip to content

Default run scripts to python3#5

Open
koriyoshi2041 wants to merge 1 commit into
XMUDeepLIT:mainfrom
koriyoshi2041:fix-script-python-default
Open

Default run scripts to python3#5
koriyoshi2041 wants to merge 1 commit into
XMUDeepLIT:mainfrom
koriyoshi2041:fix-script-python-default

Conversation

@koriyoshi2041

Copy link
Copy Markdown

Summary

  • default both run-script templates to python3 when PYTHON is unset
  • keep existing behavior for users who export a custom PYTHON value

Verification

  • bash -n code/run_index.sh
  • bash -n code/run_retrieval_test.sh
  • git diff --check
  • env -u PYTHON OPENAI_API_KEY=dummy bash code/run_index.sh now proceeds past shell expansion and stops at missing local dependency igraph in my disposable environment
  • env -u PYTHON bash code/run_retrieval_test.sh now proceeds past shell expansion and stops at missing local dependency igraph in my disposable environment

I did not run the full indexing/retrieval pipelines because dependencies, local model paths, and endpoint credentials were intentionally not configured for this small shell-template change.

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.

1 participant