[Get Familiar]: Improve logging about omrchecker HOT 11 CLOSED

Search globally for print(" and selectively edit above outputs from the screenshots @bilal-aamer

from omrchecker.

Hey, I would like to work on this

from omrchecker.

@pixalquarks sure! have a look at the code and discuss your approach on discord with me. Then I can assign it to you :)

from omrchecker.

Use colorama for color encoding. Opening and closing brackets () and [] may be colored indicatively. Errors/Warnings etc also can be colored differently.

from omrchecker.

Please state the relevent working path for 3.
@Udayraj123

from omrchecker.

Use colorama for color encoding. Opening and closing brackets () and [] may be colored indicatively. Errors/Warnings etc also can be colored differently.

I was thinking of using rich, it directly supports logging, without using ansi tags. Plus it also has properly formatted traceback. Please look into it and lemme know if I should go with rich or stick to colorama

from omrchecker.

Please assign the issue to me! @Udayraj123 !

from omrchecker.

Use colorama for color encoding. Opening and closing brackets () and [] may be colored indicatively. Errors/Warnings etc also can be colored differently.

I was thinking of using rich, it directly supports logging, without using ansi tags. Plus it also has properly formatted traceback. Please look into it and lemme know if I should go with rich or stick to colorama

Lets go with the colorama for now as it gives a more friendly looking logs in the console @Udayraj123

from omrchecker.

@bilal-aamer we shall evaluate them first as both seem useful. I've made this discussion here: https://discord.com/channels/590134763784896514/1025076621239140453/1026944631709126666

It seems rich will give us more valuable outputs as it also supports inbuilt log levels. @pixalquarks how about creating a sample log for above screenshot?

one more ref: https://www.libhunt.com/compare-rich-vs-colorama

from omrchecker.

Please assign the issue to me! @Udayraj123 !

@bilal-aamer have you run the samples as we discussed? Start a discussion on discord and propose your changes in order to get assigned.

Don't worry I can create a new ticket as this ticket seems to be broader and can be broken down into parts. especially point no. 3

from omrchecker.

Can anyone share their PRs here? @bilal-aamer @pixalquarks

from omrchecker.