Hotfix/gui buttons #95
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Clean separation of simulator and gui settings.
HoverLabel class was using python2 super invocations. This has now been updated to use the python3 style invocation.
Mostly shortening names for easy typing
Scrolling on plot items would trigger the Node dialog which was annoying. Now fixed to register a single mouse left-click.
Looks better anyways and fixes some positional bugs with MacOS
This will be used to handle all default settings for the GUI.
This class is responsible for creating menus
Patch includes callback functions for proper plot menu behavior
-Set a default file path for `load_license_text` -Added messages when the license file is unavailable or error occurs
# Conflicts: # gui_scripts/gui_helpers/menu_helpers.py
# Conflicts: # data_scripts/structure_data.py # gui_scripts/gui_helpers/general_helpers.py # gui_scripts/gui_helpers/menu_helpers.py # run_gui.py
Renamed the simulator to "FUSION Simulator" across the UI. Introduced a new About dialog with tabs for application info, license text (loaded dynamically with error handling), and space for future functionality. Refactored associated action methods for better clarity and integration. Adjusted stop button functionality.
Refactor simulation logging to send output messages to a shared log queue instead of printing to the terminal. Each simulation process now retrieves the log_queue from its configuration (set in MainWindow.set_simulation_config) and uses log_queue.put() to send log messages.
Properly unpack tuple
-new line spacing in output text -resizable topology display
ryanmccann1024
commented
Jun 5, 2025
Collaborator
Author
ryanmccann1024
left a comment
There was a problem hiding this comment.
This should contain the changes in the progress bar branch as Nathan incorporated them here. We should be careful about this and either merge the two before going into develop or ignore the progress bar branch if everything has been incorporated here correctly.
bozo-bud
reviewed
Jun 5, 2025
bozo-bud
reviewed
Jun 5, 2025
Collaborator
Author
|
@bozo-bud (Ted) to lead this branch. |
Collaborator
Author
|
@ryanmccann1024 To lead this branch. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fix GUI buttons that were not functioning properly and update GUI documentation.
Fixes # (issue)
Issue with GUI buttons but no formal issue number was created.
Type of Change
This change does relate to the GUI.
How has this change been tested?
The GUI does not have formal tests at this point in time.
Test Configuration:
Does change comply with project standards and guideline?
Standards and guidelines can be found on the Project Homepage.