Skip to content

feat(doc): add tool-vqa and tool-reasoning doc#42

Merged
ntudy merged 5 commits intoMiroMindAI:miroflow-v0.3from
xuan-dong-shanda:miroflow-v0.3
Sep 18, 2025
Merged

feat(doc): add tool-vqa and tool-reasoning doc#42
ntudy merged 5 commits intoMiroMindAI:miroflow-v0.3from
xuan-dong-shanda:miroflow-v0.3

Conversation

@xuan-dong-shanda
Copy link
Contributor

Describe this PR

Checklist for PR

Must Do

  • Write a good PR title and description, i.e. feat(agent): add pdf tool via mcp, perf: make llm client async and fix(utils): load custom config via importlib etc. CI job check-pr-title enforces Angular commit message format to PR title.
  • Run make precommit locally. CI job lint enforce ruff default format/lint rules on all new codes.
  • Run make pytest. Check test summary (located at report.html) and coverage report (located at htmlcov/index.html) on new codes.

Nice To Have

  • (Optional) Write/update tests under /tests for feat and test PR.
  • (Optional) Write/update docs under /docs for docs and ci PR.

uv run main.py prepare-benchmark get browsecomp-zh-test
uv run main.py prepare-benchmark get hle
uv run main.py prepare-benchmark get xbench-ds
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

plz fix this

@ntudy
Copy link
Contributor

ntudy commented Sep 18, 2025

The introduction of ENV VAR is confusion. These are the environment variables passed through python code, not the one set in .env.

Maybe add more introduction to tell how to set up in .env and .yaml

@ntudy ntudy merged commit e5c92d0 into MiroMindAI:miroflow-v0.3 Sep 18, 2025
1 check passed
Zhudongsheng75 pushed a commit to open-compass/MiroFlow that referenced this pull request Dec 27, 2025
* add new tools doc, support xbench-ds benchmark preparation

* docs(prepare-benchmark): add xbench-ds

* make doc clearer
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