search

idaholab / TMAP8

Tritium Migration Analysis Program, Version 8
https://mooseframework.inl.gov/TMAP8/
GNU Lesser General Public License v2.1
15 stars 21 forks source link

Add a GUI script to the fuel cycle example #192

Closed ke7kto closed 1 month ago

ke7kto commented 1 month ago

Closes #191, also terminates the example if the tritium in storage ever reaches zero.

ke7kto commented 1 month ago

I wasn't sure the right syntax to point to the scripts directory, where I threw the GUI, from the documentation. That may need to be fixed.

moosebuild commented 1 month ago

Job Documentation, step Sync to remote on fcb6f11 wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild commented 1 month ago

Job Coverage, step Generate coverage on fcb6f11 wanted to post the following:

Coverage

Coverage did not change

Full coverage report

This comment will be updated on new commits.

cticenhour commented 1 month ago

I see we've got the script tested now - great!

I did show this to PC late last week and noticed that it doesn't handle differing screen resolutions well. It was looking great on my ultrawide display at work, but on my local MacBook display or on our conference room display, it was cutting off the lower parameter adjustment fields, the run buttons, the bottom portion of the rendered plot, as well as the figure view adjustment buttons. I then wasn't able to re-size or refresh this after the window completed rendering for the first time.

How easy or difficult would this be to fixup/add and make this more flexible for more systems?

ke7kto commented 1 month ago

I've made some changes to address the resizing issue. I don't think we'll be able to get a truly modern interface for this as long as we're using tkinter, but if we need to move to an HTML/CSS based GUI that'll be a weekend project.

loganharbour commented 1 month ago

This broke moose next https://civet.inl.gov/job/2524190/

loganharbour commented 1 month ago

Skipped for now in https://github.com/idaholab/TMAP8/pull/208 until a determination is made on how to handle this with installs