Conversation
Moved a few things around I got this to work on my Windows 10 machine seems like Spawn() does the trick!!
|
Thanks - but did you mean to make the code changes to the |
|
Hey, yes it was supposed to be windows sorry, I had tested it on a offline copy |
|
Yes please :)
…On Sun, 8 Jan 2023 at 16:43, Juanes Paz ***@***.***> wrote:
Hey, yes it was supposed to be windows sorry, I had tested it on a offline
copy
Do you want me to correct it?
—
Reply to this email directly, view it on GitHub
<#129 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHSVKWFKGQXY3KOQ35FLXB3WRLVCHANCNFSM6AAAAAATUR4LUQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
This reverts commit 780976a.
Hi, sorry for the last PR updated!!
|
Passing all the checks; that's good. One question though, what is the difference between the command |
the separate tags didn't make a difference on the original you had .split the message in args(), you also had the /C there, I am not sure why it is needed :( |
|
I found what I've just updated the file very slightly. Are you able to confirm that the latest commit (1818494) works on your computer? Then I can merge this. |
|
Hey, that is a good find super clear now!! thank you for sharing... |
|
Great, thanks for that :) I'll release v0.8.4 with this fix once at least #126 is merged. |
* plot.show() fix Moved a few things around I got this to work on my Windows 10 machine seems like Spawn() does the trick!! * Revert "plot.show() fix" This reverts commit 780976a. * fix plot.show() Windows Hi, sorry for the last PR updated!! * fix command line args * Update CHANGELOG.md * Update CHANGELOG.md Co-authored-by: Michael Freeborn <31806808+mfreeborn@users.noreply.github.com>
Moved a few things around I got this to work on my Windows 10 machine seems like Spawn() does the trick!!