Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Runspaces for Microwin #2731

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

MyDrift-user
Copy link
Contributor

@MyDrift-user MyDrift-user commented Sep 13, 2024

Type of Change

  • Refactoring

Description

Use Runspaces in Microwin. For main process of modifying ISO should get divided into multiple runspaces to work at the same time and be faster.

  • started with the basics but will temporarily leave the PR like this bc of the currently opened PRs for Runspace and for Auto ISO download.
  • Many functions will need to be renamed due to them having to be added into the runspaces

Testing

Invoke-WPFGetIso:

  • rename Copy-Files to "Invoke-WinUtilCopyFiles" so they get loaded into runspaces
  • move elements in WPFGetIso
  • wrap most of the content into runspace
  • fix some runspace issues in WPFGetIso
  • starting with $sync.MicrowinWindowsFlavors.Items.Clear() (the part where it tries to update UI) I have not made it work yet.

ISO Modification Process:

  • Not started yet

Impact

  • UI can be used while using Microwin
  • Microwin Process runs faster

Issue related to PR

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@MyDrift-user MyDrift-user reopened this Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Runspaces for Microwin
2 participants