Closed antimora closed 1 year ago
Hey, I was browsing reddit and found this project when looking for a Rust project to contribute to. I would love to help out. Has this issue been resolved yet?
Hi @NicBonetto , welcome to the project.
This has not been resolved. Your contribution is highly appreciated.
I filed this ticket originally. I thought the training charts are not that clear and could be improved.
@nathanielsimard and I are here to review your changes or can talk on discord to ask for help.
@NicBonetto Don't be afraid to come talk on Discord about how you want to tackle the issue; we can help. :)
I think we should replace the textplot
with termplot library because textplot
is not updated anymore and uses nom
version 1 which gives a warning when building burn.
I've just started to play around with the examples, and may need some help with initial setup. Is this a good place to ask or should we move the discussions over to discord? (sorry in advanced if these are stupid questions).
I've just started to play around with the examples, and may need some help with initial setup. Is this a good place to ask or should we move the discussions over to discord? (sorry in advanced if these are stupid questions).
Please do not worry about asking questions. We are all learning here and often we are not sure either.
I would recommend to try discord for back and worth questions/answers. We are all on it and it's not busy over there.
But I would suggest you keep updating the ticket to have a record of the discussion and new findings. It will help others learn as well.
I think we should replace the current implementation with something like ratatui https://github.com/ratatui-org/ratatui. It would be better than the actual implementation and seems much more maintained, potentially avoiding security vulnerabilities.
Describe the bug
The plots are not clear and have too many dots.
To Reproduce Run training of
examples/mnist
Expected behavior It would be if a different markers are used to plot.
Screenshots
Desktop (please complete the following information):