Removing Rich

[itsmeadarsh2008]

Rich is too heavy with bloatware in the dependency. Would it be a better idea to make a progress bar using tqdm only because using rich doesn't make any sense with web apps in the production? It will be a good idea if we replace it with them and keep it simple. You can convert it to an issue if you want. Here are some example sentences. (No Boxes, Just Logs And These Sentences)

1. ⚡ Compiling ETA: {seconds}
2. 🚧 Stopping the server
3. 🏓 Frontend is playing ping-pong on {port}
4. 🖥 Backend is serving food on {port}
5. Your server is ready to go 🚀

[IamAdiSri]

Hi, I would like to work on this and was wondering about the extent to which this will change things. Are we completely removing rich from the codebase? Should the ETA be a dynamically changing number or should it just be printed once?

[ndb4263]

Hey, I would like to work on this bounty.

[ai-naymul]

@itsmeadarsh2008 , What's should I use in case of the logging? Should I use the python's print function or the tqdm's write function?

[ai-naymul]

It just replaces the progress bar. Try to remove the rich completely. Work on your own I guess.

Hi @itsmeadarsh2008 ,

I think there are no test cases neededd for this migration cause there were not also any test cases for the rich dependency and its just to show the progress bar, Am I right bro?

[itsmeadarsh2008]

Good evening. Test cases are needed whenever you change the internal structure of a project. Since, it's the CLI which shows the progress of compilation. You need to check whether all your test cases have passed or not. By the way, Try to search for rich all over the project make necessary migrations to another one (less-sized) like colorama, tqdm etc.
https://github.com/search?q=org%3Areflex-dev%20rich&type=code

[kraanzu]

Hey @itsmeadarsh2008, Is this issue still open? The author hasn't made any updates for a month :)

[itsmeadarsh2008]

@kraanzu yes, the issue is still open. you can contribute, or make a new PR instead.

[kraanzu]

Ok cool. Thanks for confirming @itsmeadarsh2008 ! :D

[dineshpabbati]

Hi,
I'd like to work on Removing Rich #3479.