Conversation
|
@G0rocks Nice little function! However, I don't think this goes in the source code of this package; it's very specific, and it doesn't need to be hard-coded into the source to work. The Alternatively, you could add it in the |
|
Thanks :) So if I understand you correctly, you mean that anybody who wants to use python-VPP to write the polar plot data into a csv file should make their own code to make that happen and that python-VPP can and should only be able to output the polar plot into the results.json file? But I can move it to UtilsMod.py and it can stay? Because I need to work with this polar after generating it and can not use the json file. Using the csv file makes it way more usable to my use case. |
|
Dear all, having it output to a csv will be very useful too
…On Thu, 19 Feb 2026, 11:29 Huldar, ***@***.***> wrote:
*G0rocks* left a comment (marinlauber/Python-VPP#50)
<#50 (comment)>
Thanks :)
And actually, until this issue
<rgleason/polar_pi#34> is dealt with then it
basically needs to be hardcoded in to work.
So if I understand you correctly, you mean that anybody who wants to use
python-VPP to write the polar plot data into a csv file should make their
own code to make that happen and that python-VPP can and should only be
able to output the polar plot into the results.json file?
But I can move it to UtilsMod.py and it can stay? Because I need to work
with this polar after generating it and can not use the json file. Using
the csv file makes it way more usable to my use case.
—
Reply to this email directly, view it on GitHub
<#50 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCUWWDQ5RDTD2ZWC7K2T3D4MWF65AVCNFSM6AAAAACVSLN7UWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTSMRWGI4DQMBVGY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Moving this to the UtilsMod.py was a little bit of a tricky thing because of the dependencies but worked eventually. Please check the new PR, approve and merge when it is ready. |
Fixes issue #45
Also contains the code from the other PR I made, I'm sure I can figure out how to change that at some point but don't feel like doing it right now.