apple / apple/GCGC

plot_pie_sum appends the builtin slice type instead of the computed slice size

Open Beginner friendly
#23 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
455
Forks
48
Avg merge
4m
Merged PRs (30d)
2

Description

src/graphing/plotting.py:110-113:
for idx, datapoints in enumerate(datapoint_groups):
pslice = sum(datapoints)
pie_slices_sizes.append(slice) # <-- appends builtin `slice`, not `pslice`
labels[idx] = labels[idx] + " : " + str(pslice)
pslice is computed and used in the label, but the list gets the builtin slice type
object. So plot.pie() receives a list of type objects rather than numbers.

Contributor guide

No contributing guide indexed for this repository

Research direction

Open src/graphing/plotting.py lines 110-113 and inspect plot_pie_sum, starting with the value appended to pie_slices_sizes. Run the plotting entry point that calls plot.pie() and verify it receives the computed numeric slice sizes while the labels still show those sums.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-visualization
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
92/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.