[Feature] Add flag to perform a single dashboard update - render cycle about reviews HOT 6 CLOSED

@apoclyps do you mean options instead?

https://click.palletsprojects.com/en/7.x/options/#options

from reviews.

Yes, thanks for making a note that options was the correct usage here.

I'll check out your WIP shortly and see if there is any way we can do this. It may be a limitation of the library being used or a feature that isn't generally supported.

from reviews.

This may not be possible using the existing classes in the rich library because of the following:

The live display will persist for the duration of the context. You can update the renderable to update the display

https://rich.readthedocs.io/en/stable/live.html#basic-usage

transient (bool, optional): Clear the renderable on exit (has no effect when screen=True). Defaults to False.

https://github.com/willmcgugan/rich/blob/f513abd35fdacccdc6fcbcc6989b5792b75ec97a/rich/live.py#L43

Setting transient=False, screen=False allows the UI to persist however it tends to flicker a lot whilst rendering. To achieve this without flickering, we might need to create another render that only performs a single render with the completed results once completed with the required settings.

from reviews.

@apoclyps do you mean options instead?

https://click.palletsprojects.com/en/7.x/options/#options

I've started working on this, but am currently facing an issue where when I quit off the main loop, the last refreshed display goes away with it and thus is useless -- I've referred to the docs regarding transient display but that's not really helpful too. Will see if there's any other way to accomplish this

from reviews.

This may not be possible using the existing classes in the rich library because of the following:

The live display will persist for the duration of the context. You can update the renderable to update the display

https://rich.readthedocs.io/en/stable/live.html#basic-usage

transient (bool, optional): Clear the renderable on exit (has no effect when screen=True). Defaults to False.

https://github.com/willmcgugan/rich/blob/f513abd35fdacccdc6fcbcc6989b5792b75ec97a/rich/live.py#L43

Setting transient=False, screen=False allows the UI to persist however it tends to flicker a lot whilst rendering. To achieve this without flickering, we might need to create another render that only performs a single render with the completed results once completed with the required settings.

I tried an idea after looking at this and it seems to be working -- i set screen= not no_reload here

from reviews.

For a larger number of repositories and pull requests the screen flickering is still an issue in #76

I've implemented an alternative solution in #78 - I would like to know your opinions on the solution.

from reviews.