WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Feat/piechart in multiplot #173

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.

Sign up for GitHub

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

Jump to bottom
Open
Tanguylo wants to merge 46 commits into
base: dev
Choose a base branch
from
Open

Feat/piechart in multiplot #173

Tanguylo wants to merge 46 commits into from

Conversation

@Tanguylo
Copy link
Collaborator

@Tanguylo Tanguylo commented Jun 23, 2022

No description provided.

Tanguylo added 20 commits June 3, 2022 14:51
@Tanguylo
Ajout class Pie
e2280d1
@Tanguylo
Start with piechart
d83ec07
@Tanguylo
feat(piechart): first piechart drawing
d5b2e4b
@Tanguylo
feat(piechart): add colors to piechart
b4514ea
@Tanguylo
feat(piechart):add pie part surface coloring on mouse over
e21e984
@Tanguylo
feat(piechart):partial code cleaning
949df9c
@Tanguylo
feat(piechart):fix a git mistake with branches
8427fef
@Tanguylo
feat(piechart):fixing last diff in core.py
e21259e
@Tanguylo
feat(piechart):remove ' /' line 931
ad59106
@Tanguylo
feat(piechart):add parts selection on click
6c494e2
@Tanguylo
feat(piechart):manage piepart paths as piepart attribute
9c83b56
@Tanguylo
feat(piechart):initialize pieparts path when instantiating when creat…
38ebcb3 
…ing parts
@Tanguylo
feat(piechart):some code cleaning
1937904
@Tanguylo
feat(piechart_in_multiplot): adds piechart case in multiplot
58cca16
@Tanguylo
feat(piechart): fix mistakes following the first 6 comments on this p…
4675ede 
…ull request
@Tanguylo
feat(piechart): fix the 5 following remarks of the PR
cd1864f
@Tanguylo
feat(piechart): fix the remaining remarks except the drawPieChart method
1f81acc
@Tanguylo
feat(piechart): take last PR remarks into account
e3371ec
@Tanguylo
feat(piechart): change type of color_to_plot_data to map
84fe419
@Tanguylo
feat(piechart): modifications following PR remarks
8bf219a
@Tanguylo Tanguylo linked an issue Jun 23, 2022 that may be closed by this pull request
Support piechart in multiplot #171
Closed
1 task
Tanguylo added 16 commits June 27, 2022 10:37
@Tanguylo
feat(piechart): Undo color_to_plot_data as a Map
87d5ef6
@Tanguylo
feat(piechart): Changes following second PR
10772ba
@Tanguylo
feat(piechart): DataSample and DataSamples type definition and some c…
34cdd69 
…hanges following the PR review
@Tanguylo
feat(piechart_in_multiplot): remove a console.log
80b5b08
@Tanguylo
feat(piechart_in_multiplot): merge piechart in piechart_in_multiplot
64c9094
@Tanguylo
feat(piechart): Changes in drawPieChart
7a5f0e6
@Tanguylo
feat(piechart): Change multiple_plot.py file
54091b1
@Tanguylo
feat(piechart_in_multiplot): Draw a PieChart in a multiplot
d2a38ee
@Tanguylo
feat(piechart_in_multiplot): Change multiple_plot.py to diagnose the …
23f045e 
…problem of offset
@Tanguylo
feat(piechart_in_multiplot): Merge origin/feat/piechart
714c0f4
@Tanguylo
feat(piechart_in_multiplot): Put some console.log in clean_view. Chec…
feed8ec 
…kpoint before changing branch
@Tanguylo
feat(piechart): Call definePieParts in PieChart constructor and not a…
0ecc993 
…nymore PlotPieChart's one and change radius to have a standard PieChart instantiated in plots with a .transform(matrix)
@Tanguylo
Merge remote-tracking branch 'origin/feat/piechart' into feat/piechar…
e704341 
…t_in_multiplot
@Tanguylo
feat(piechart_in_multiplot): Center PieChart in multiplots
b37595e
@Tanguylo
feat(piechart_in_multiplot): Center even the last piechart of a multi…
4dc6905 
…plot
@Tanguylo
feat(piechart_in_multiplot): Remove a useless line
a5dfc2b
@Tanguylo Tanguylo marked this pull request as ready for review June 28, 2022 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xavier-ferry xavier-ferry Awaiting requested review from xavier-ferry

@GhislainJ GhislainJ Awaiting requested review from GhislainJ

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

status: stopped

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support piechart in multiplot

2 participants

@Tanguylo