[PR #382] [CLOSED] Pie chart (multivalue donut) #374

Closed
opened 2026-03-03 16:24:08 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mum4k/termdash/pull/382
Author: @VincenzoManto
Created: 8/23/2025
Status: Closed

Base: masterHead: master


📝 Commits (10+)

  • bd606b2 Implement Pie widget with options and validation; add demo and tests
  • c462e7d Add Pie widget documentation and demo instructions to README
  • 253fc26 Refactor options.go to include necessary imports for option validation
  • 444eb92 Refactor Pie widget to remove validation for values and adjust color handling in options
  • 36fe9cb Remove unused imports and change DefaultColors to a var declaration in options.go
  • fdcf8bb Remove color validation from options as it is not required
  • 5dc0455 Refactor pie widget tests and demo to remove unused imports and simplify value handling
  • 4777e0c Remove unused import from pie_test.go
  • ac1f3ce Set default colors in Pie widget if none are provided
  • c44e7a1 Remove test case for empty colors in pie chart

📊 Changes

5 files changed (+429 additions, -0 deletions)

View changed files

📝 README.md (+12 -0)
widgets/pie/options.go (+50 -0)
widgets/pie/pie.go (+167 -0)
widgets/pie/pie_test.go (+135 -0)
widgets/pie/piedemo/piedemo.go (+65 -0)

📄 Description

A pie chart implementation used in Datacmd as you can see below.

If something must be changed in tests or options definitions, let me know

image

🔄 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/mum4k/termdash/pull/382 **Author:** [@VincenzoManto](https://github.com/VincenzoManto) **Created:** 8/23/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`bd606b2`](https://github.com/mum4k/termdash/commit/bd606b27a76640925c4668a1e875dffe1725982e) Implement Pie widget with options and validation; add demo and tests - [`c462e7d`](https://github.com/mum4k/termdash/commit/c462e7d4bb543b7534bd5be60453c22fcf729418) Add Pie widget documentation and demo instructions to README - [`253fc26`](https://github.com/mum4k/termdash/commit/253fc26634346f6e39dfe8f2cb08835a7cda2bd8) Refactor options.go to include necessary imports for option validation - [`444eb92`](https://github.com/mum4k/termdash/commit/444eb92a7411d0be59da388ac03ccbe1af5b116f) Refactor Pie widget to remove validation for values and adjust color handling in options - [`36fe9cb`](https://github.com/mum4k/termdash/commit/36fe9cbcbcff02b11d59826ef916358543184564) Remove unused imports and change DefaultColors to a var declaration in options.go - [`fdcf8bb`](https://github.com/mum4k/termdash/commit/fdcf8bb8c3577b385464a623f1f02279be984c18) Remove color validation from options as it is not required - [`5dc0455`](https://github.com/mum4k/termdash/commit/5dc045552966cde6b2bf48cfebcd619aa0e9744d) Refactor pie widget tests and demo to remove unused imports and simplify value handling - [`4777e0c`](https://github.com/mum4k/termdash/commit/4777e0c81226657f99764aaab9ce794f51eaece1) Remove unused import from pie_test.go - [`ac1f3ce`](https://github.com/mum4k/termdash/commit/ac1f3cee4581a66a3f128050c9ed22c0a713a7e9) Set default colors in Pie widget if none are provided - [`c44e7a1`](https://github.com/mum4k/termdash/commit/c44e7a12da08cf4882a434209a7f6e607eec68c1) Remove test case for empty colors in pie chart ### 📊 Changes **5 files changed** (+429 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+12 -0) ➕ `widgets/pie/options.go` (+50 -0) ➕ `widgets/pie/pie.go` (+167 -0) ➕ `widgets/pie/pie_test.go` (+135 -0) ➕ `widgets/pie/piedemo/piedemo.go` (+65 -0) </details> ### 📄 Description A pie chart implementation used in [Datacmd](https://github.com/VincenzoManto/Datacmd) as you can see below. If something must be changed in tests or options definitions, let me know <img width="1473" height="623" alt="image" src="https://github.com/user-attachments/assets/5d45cfa8-1811-489f-ad77-f2d3b61ad9ed" /> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 16:24:08 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/termdash#374
No description provided.