-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Default example doesn't work #1
Comments
Hey there, what OS are you running on? I haven't looked at this in awhile, and I should've mentioned... I've only tested this in CMD |
Mac, though I could try it on my linux machine too. |
Yeah, same on linux.
|
same issue almost two years later unresolved. Im fixing it ill send a pull request. I am attempting to fix this for Linux/Unix based systems. I have narrowed down some the the issues and resolved the specified path issue. If you observe |
Fixed it! The only issue is color. In linux the colors will not work in the default examples I can sort this out at a later date. There is also the minor issue of some of the examples not containing the proper sub directories mentioned in the repr , ie ascii and color folders. This is a neat little program and I hope Linux/Unix users can use it now. :-) Pull request sent |
Thanks! I haven’t looked at this project in a while but feel free to fork
and merge at your will
…On Wed, 20 Dec 2023 at 10:37 AM, joshjetson ***@***.***> wrote:
Fixed it! The only issue is color. In linux the colors will not work in
the default examples I can sort this out at a later date. There is also the
minor issue of some of the examples not containing the proper sub
directories mentioned in the repr , ie ascii and color folders. This is a
neat little program and I hope Linux/Unix users can use it now. :-)
—
Reply to this email directly, view it on GitHub
<#1 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AR553ZVFPGUHDN7H5L7SXJLYKICKNAVCNFSM5MBAPSEKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBWGM2TANRZGA4A>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
I sent it your way my bro, pending your approval. :-) Cheers! |
Did you not want to merge into your branch ? You could check the diff I did not modify your original I just included a new file which is basically your application but modified to work on linux systems. There were some other issues too that I addressed an am surprised worked on a windows system. You were attempting to create instance attributes outside of the dunder init method. I would appreciate being able to be a contributior to the project by you merging the pull request. |
Am I doing something wrong? I'm running the code exactly as in the readme.
Thanks for a cute and creative library!
The text was updated successfully, but these errors were encountered: