[PR #7143] [MERGED] Clean up package layout #38200

Closed
opened 2026-04-22 22:52:37 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/7143
Author: @dhiltgen
Created: 10/8/2024
Status: Merged
Merged: 10/17/2024
Merged by: @dhiltgen

Base: mainHead: go_server_layout


📝 Commits (1)

  • 70a2838 Rename gpu package discover

📊 Changes

33 files changed (+94 additions, -94 deletions)

View changed files

📝 discover/amd_common.go (+1 -1)
📝 discover/amd_hip_windows.go (+1 -1)
📝 discover/amd_linux.go (+1 -1)
📝 discover/amd_windows.go (+1 -1)
📝 discover/cpu_common.go (+1 -1)
📝 discover/cuda_common.go (+1 -1)
📝 discover/gpu.go (+1 -1)
📝 discover/gpu_darwin.go (+1 -1)
📝 discover/gpu_info.h (+0 -0)
📝 discover/gpu_info_cudart.c (+0 -0)
📝 discover/gpu_info_cudart.h (+0 -0)
📝 discover/gpu_info_darwin.h (+0 -0)
📝 discover/gpu_info_darwin.m (+0 -0)
📝 discover/gpu_info_nvcuda.c (+0 -0)
📝 discover/gpu_info_nvcuda.h (+0 -0)
📝 discover/gpu_info_nvml.c (+0 -0)
📝 discover/gpu_info_nvml.h (+0 -0)
📝 discover/gpu_info_oneapi.c (+0 -0)
📝 discover/gpu_info_oneapi.h (+0 -0)
📝 discover/gpu_linux.go (+1 -1)

...and 13 more files

📄 Description

This shuffles code around to try to clean up the package layout a bit in preparation for removal of the submodule.

  • ./gpu/ -> ./discover/ (simple rename) : clearer name since this has evolved to cover more than just GPUs

(llm package cleanup deferred to a follow up...)


🔄 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/7143 **Author:** [@dhiltgen](https://github.com/dhiltgen) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/17/2024 **Merged by:** [@dhiltgen](https://github.com/dhiltgen) **Base:** `main` ← **Head:** `go_server_layout` --- ### 📝 Commits (1) - [`70a2838`](https://github.com/ollama/ollama/commit/70a283853dc57b9c95fc009636e67fdd1c781c1f) Rename gpu package discover ### 📊 Changes **33 files changed** (+94 additions, -94 deletions) <details> <summary>View changed files</summary> 📝 `discover/amd_common.go` (+1 -1) 📝 `discover/amd_hip_windows.go` (+1 -1) 📝 `discover/amd_linux.go` (+1 -1) 📝 `discover/amd_windows.go` (+1 -1) 📝 `discover/cpu_common.go` (+1 -1) 📝 `discover/cuda_common.go` (+1 -1) 📝 `discover/gpu.go` (+1 -1) 📝 `discover/gpu_darwin.go` (+1 -1) 📝 `discover/gpu_info.h` (+0 -0) 📝 `discover/gpu_info_cudart.c` (+0 -0) 📝 `discover/gpu_info_cudart.h` (+0 -0) 📝 `discover/gpu_info_darwin.h` (+0 -0) 📝 `discover/gpu_info_darwin.m` (+0 -0) 📝 `discover/gpu_info_nvcuda.c` (+0 -0) 📝 `discover/gpu_info_nvcuda.h` (+0 -0) 📝 `discover/gpu_info_nvml.c` (+0 -0) 📝 `discover/gpu_info_nvml.h` (+0 -0) 📝 `discover/gpu_info_oneapi.c` (+0 -0) 📝 `discover/gpu_info_oneapi.h` (+0 -0) 📝 `discover/gpu_linux.go` (+1 -1) _...and 13 more files_ </details> ### 📄 Description This shuffles code around to try to clean up the package layout a bit in preparation for removal of the submodule. - `./gpu/` -> `./discover/` (simple rename) : clearer name since this has evolved to cover more than just GPUs (llm package cleanup deferred to a follow up...) --- <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-22 22:52:37 -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#38200