Adding Piechart support

[baszach]

Greetings,
I added the Pie trace and some needed elements (e.g. Direction or PieTitleFont). Is the written code going in the right direction? Also, should I write a test in the DocumentationTests class?

[alexarchambault]

Thanks! You might want to add things in DocumentationTests, yes, and see how it goes, by adding basic/pie in this list. This will try to run these examples in the tests.

I also see you added 3 enumeration (Direction, PieTextPosition, PieTitlePosition). You need to state these are enumerations, by adding a line for each of them around here. That allows them to be encoded / decoded as strings.

[baszach]

Hello! I proceeded as you suggested and implemented a test, registered the enums etc etc...

When I run the test DocumentationTests it fails for the pie chart. The error says Decoding data: Found extra fields: textinfo, color. I think it's because I haven't implemented every possible field from here. Now my questions are: Why don't these tests fail for other Traces, as they don't implement each field either? And is the current state of the PR sufficient or is something missing?

Thank you in advance :)))))

[baszach]

Hello @alexarchambault
did you manage to take a look at my last commits? It would be great if you could give me feedback so I can make the changes, if necessary.
Thanks :)

[baszach]

Hello again,
can this PR be merged or is there anything missing? Would be cool to be able to use that pie chart :)
@alexarchambault thanks

[tflucke]

@alexarchambault I would also like to express interest in getting this merged in.