Skip to content

Added download script and uv installation structions#20

Open
coezbek wants to merge 3 commits intoxiaomi-research:mainfrom
coezbek:main
Open

Added download script and uv installation structions#20
coezbek wants to merge 3 commits intoxiaomi-research:mainfrom
coezbek:main

Conversation

@coezbek
Copy link

@coezbek coezbek commented Mar 24, 2025

  • Added a download script for mini-test mmau files.
  • Added instructions for dependency installation with uv
  • Added the inference instructions from hugging face

@jimbozhang
Copy link
Contributor

Cool~ @RicherMans could you review this?

@RicherMans
Copy link

Hey,
I tested the code running with:

Kernel 5.15.122
Python 3.10.14
uv 0.5.4
GNU bash, version 5.2.37

Works flawlessly, output after

git clone https://github.com/xiaomi-research/r1-aqa
git  remote add pr https://github.com/coezbek/r1-aqa
git pull main pr
./download_mmau.sh
uv init
uv add -r requirements.txt
uv run inference.py

README.md just needs to be updated accordingly.

@coezbek
Copy link
Author

coezbek commented May 9, 2025 via email

@RicherMans
Copy link

Hey @coezbek ,
thanks, yeah also works with uv sync I just followed the README instructions :).
Thanks for the PR, but can you please do a pull from our current main branch?
Sorry that we replied this late, but the README has currently a merge conflict, since we updated some results.

Kind regards,
Heinrich

@coezbek
Copy link
Author

coezbek commented May 9, 2025

@RicherMans I merged the README change, but haven't tested the download script again.

@frankenliu frankenliu closed this May 9, 2025
@coezbek
Copy link
Author

coezbek commented May 9, 2025

@frankenliu Isn't it working? I can look back into it. I thought @RicherMans tested it and only wanted me resolve the merge conflict in the README.

@frankenliu frankenliu reopened this May 9, 2025
@frankenliu
Copy link
Collaborator

@coezbek I have reviewed the PR. Can you tell me why add the inference.py? This file i think the same as src/test_mmau.py, can you use the test_mmau.py?

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.

4 participants