Skip to content

Refactor/scaffold gui#110

Merged
ryanmccann1024 merged 6 commits intorefactor/scaffoldfrom
refactor/scaffold_gui
Aug 13, 2025
Merged

Refactor/scaffold gui#110
ryanmccann1024 merged 6 commits intorefactor/scaffoldfrom
refactor/scaffold_gui

Conversation

@bozo-bud
Copy link
Copy Markdown
Contributor

@bozo-bud bozo-bud commented Aug 8, 2025

Description


This PR refactors the GUI's pathing and directory topology to improve readability and scalability of the project. It additionally brings the GUI's run process in line with other simulation types.

Fixes # (issue)

Fixes: The GUI can now be run through the Run pipeline specific to refactored modules.

Type of Change


Please delete options that are not relevant. Please specify if change
applies to GUI.

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring, compatibility patching

Testing

All changes were tested by:
* Running of the GUI through CLI using:
git push --set-upstream origin refactor/scaffold_gui

…/scaffold

# Conflicts:
#	fusion/helper_scripts/sim_helpers.py
Moved files to more scalable directories. Changed naming conventions to be more consistent and readable.
Broke up multi-class files into multiple single class files. Added skeleton files for future refactor.
@bozo-bud bozo-bud self-assigned this Aug 8, 2025
@ryanmccann1024
Copy link
Copy Markdown
Collaborator

We might want to resolve those conflicts first so the rest of the checks can run.

I don't think any of the pipelines ran on the pull request, only the pull request target

Copy link
Copy Markdown
Collaborator

@ryanmccann1024 ryanmccann1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if the pipelines pass after resolving the merge conflict this is good to go.

Also now please point this at refactor/scaffold. You'll have to do a quick merge beforehand most likely.

@bozo-bud bozo-bud changed the base branch from refactor/rl-cli to refactor/scaffold August 12, 2025 19:33
@ryanmccann1024 ryanmccann1024 merged commit 195eb3e into refactor/scaffold Aug 13, 2025
4 of 6 checks passed
ryanmccann1024 added a commit that referenced this pull request Aug 14, 2025
@ryanmccann1024 ryanmccann1024 deleted the refactor/scaffold_gui branch January 19, 2026 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants