search

ucsf-deb / bifs

Bayesian Image Analysis in Fourier Space
Other
0 stars 4 forks source link

put the GUI in bifs_gui #21

Closed RossBoylan closed 2 years ago

RossBoylan commented 4 years ago

Low priority, but ... bifs_gui delivers a lot of its error messages via print. It should instead be popping up graphical widgets with the appropriate message and symbol (error/warning/info...). Qt provides functions to do this, so it shouldn't be too painful.

A quick and dirty solution would be to redefine print within bifs_gui to invoke one of those functions, but just doing a search and replace is likely safer.

kyoung21b commented 4 years ago

Yep, another vestige of the quick dirty development that hasn’t been replaced, though I do recall looking at the QT error widgets and realized that a little care is required re. implementing those; there is some history baked into them, such that there are different mechanisms available and choosing one and staying consistent requires a little work.

— K

Sent from this stupid iPhone

On Nov 29, 2019, at 1:11 PM, RossBoylan notifications@github.com wrote:

Low priority, but ... bifs_gui delivers a lot of its error messages via print. It should instead be popping up graphical widgets with the appropriate message and symbol (error/warning/info...). Qt provides functions to do this, so it shouldn't be too painful.

A quick and dirty solution would be to redefine print within bifs_gui to invoke one of those functions, but just doing a search and replace is likely safer.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

RossBoylan commented 2 years ago

Basically done by the 0.9.4 release