Skip to content

update doc#120

Merged
weixlu merged 1 commit into
liebharc:mainfrom
weixlu:doc
Jul 15, 2026
Merged

update doc#120
weixlu merged 1 commit into
liebharc:mainfrom
weixlu:doc

Conversation

@weixlu

@weixlu weixlu commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator
  • README.md: after Use standardized (PEP 517) dependency syntax in pyproject.toml #114 , gpu dependency should be installed via --extras gpu
  • Training.md: update SMB benchmark result on Run 426. smb_new.log
  • Benchmark.md: That's what I want to discuss. Should we update it as well? And besides, I notice benchmark results are uploaded to release page, so maybe I should do the same this time. If so, then we need to think about how to give it a name for each run.

@liebharc

Copy link
Copy Markdown
Owner

Regarding, Benchmark.md and our release page. That's a good point. I think I want to keep things lean and not add too much process. In that sense I would occasionally update the numbers in Benchmark.md and upload new dbs to the release page, but not with every model update. It might be something to consider with every new minor version release. Training.md then remains the main spot to track results and this way we should also be able to spot regressions. And Benchmark.md and the DB releases have the purpose of giving people an idea what performance they can expect from homr.

Then it might however be good to add a note to Benchmark.md that these aren't necessarily the newest results.

@weixlu

weixlu commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

Sure, I also think we don't have to add too much process. Then this PR may seem fine.

@liebharc

Copy link
Copy Markdown
Owner

Yes, feel free to hit the merge button!

@weixlu
weixlu merged commit ff226cb into liebharc:main Jul 15, 2026
1 check passed
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