[PR #4058] [MERGED] fix: store accurate model parameter size #11366

Closed
opened 2026-04-12 23:28:48 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/4058
Author: @BruceMacD
Created: 4/30/2024
Status: Merged
Merged: 5/7/2024
Merged by: @BruceMacD

Base: mainHead: brucemacd/format_param_count


📝 Commits (2)

  • 45c7f81 fix: store accurate model parameter size
  • d5d821e Update format/format.go

📊 Changes

2 files changed (+48 additions, -6 deletions)

View changed files

📝 format/format.go (+14 -6)
format/format_test.go (+34 -0)

📄 Description

  • add test for number formatting
  • fix bug where 1B and 1M were not stored correctly
  • display 2 decimal points for million param sizes
  • display 1 decimal point for billion param sizes

This human conversion is displayed as the parameter size on ollama.com, so it should be in the standard format that model parameter sizes are measured.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/4058 **Author:** [@BruceMacD](https://github.com/BruceMacD) **Created:** 4/30/2024 **Status:** ✅ Merged **Merged:** 5/7/2024 **Merged by:** [@BruceMacD](https://github.com/BruceMacD) **Base:** `main` ← **Head:** `brucemacd/format_param_count` --- ### 📝 Commits (2) - [`45c7f81`](https://github.com/ollama/ollama/commit/45c7f818303e8c0e35c2c4572b10d2ebe489b829) fix: store accurate model parameter size - [`d5d821e`](https://github.com/ollama/ollama/commit/d5d821e11f47790f540692d2a0d2b9b8c1283dbf) Update format/format.go ### 📊 Changes **2 files changed** (+48 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `format/format.go` (+14 -6) ➕ `format/format_test.go` (+34 -0) </details> ### 📄 Description - add test for number formatting - fix bug where 1B and 1M were not stored correctly - display 2 decimal points for million param sizes - display 1 decimal point for billion param sizes This human conversion is displayed as the parameter size on ollama.com, so it should be in the standard format that model parameter sizes are measured. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-12 23:28:48 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#11366