Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: display model storage size #7877

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

keturn
Copy link
Contributor

@keturn keturn commented Apr 3, 2025

How much disk space is each model taking up? Inquiring minds want to know.

Filesystem management tools could tell us this too, but if users go mucking around in the models filesystem and delete things there (outside of the app) then the database no longer reflects reality.

Related Issues / Discussions

https://discord.com/channels/1020123559063990373/1357425455816507452

QA Instructions

  • look at model list in model manager, you should see file sizes listed in gigabytes
  • file size should also be listed in the model detail panel
  • try installing a model

Merge Plan

There is a database migration!

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions bot added api python PRs that change python files backend PRs that change backend files services PRs that change app services python-tests PRs that change python tests labels Apr 3, 2025
@keturn keturn force-pushed the feat/show_model_size branch 2 times, most recently from 5ebccb3 to b81e0f5 Compare April 4, 2025 21:54
@github-actions github-actions bot added the frontend PRs that change frontend files label Apr 4, 2025
@keturn keturn force-pushed the feat/show_model_size branch from 1075fbd to 8cd6940 Compare April 4, 2025 23:44
@keturn
Copy link
Contributor Author

keturn commented Apr 4, 2025

Okay, this technically works now.

Do we want to disambiguate "size"? Call it "storage size" or "disk size" or something to distinguish from RAM size or pixel size?

@hipsterusername
Copy link
Member

File Size? Model Size?

I think it probably is clearest 🤔

@keturn keturn force-pushed the feat/show_model_size branch from 3c00786 to 4ecc699 Compare April 5, 2025 22:17
@github-actions github-actions bot added the frontend-deps PRs that change frontend dependencies label Apr 9, 2025
Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

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

Nice! I made a couple small changes:

  • Add and use the filesize library to handle formatting the file size.
  • Tweak the UI layout.
  • Rebase on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api backend PRs that change backend files frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies python PRs that change python files python-tests PRs that change python tests services PRs that change app services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants