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

Update install_latest_blue_onyx.ps1 #36

Merged
merged 13 commits into from
Dec 28, 2024
Merged

Update install_latest_blue_onyx.ps1 #36

merged 13 commits into from
Dec 28, 2024

Conversation

DeFlanko
Copy link
Contributor

  • Updated the PATH environment be the System of Machine, this will allow for automation with Task Scheduler to run the bat files as a Service Account. This will require that the Powershell be ranas Administrator!

image

  • Updated install options to include a folder selection for the end user to install the app.

image

  • Added 4th bat file for Large models. "blue_onyx_start_server_large_models.bat"

  • Updated final color script -- now with more colors! (helps with readability)

image

Updated install options to include a popup folder selection to install the app.

Added 4th bat file for Large models.

Updated final color script to break up the Green Noise. 

Signed-off-by: DeFlanko <[email protected]>
Added user inputbox for GPU selection, the value of that selection then gets put into the BAT files. 

Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
Get a list of available GPUs 

We also want to exclude "RDP Displays" and Sort the list like how it is in Task Manager so the Index is correct. 

Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
@DeFlanko
Copy link
Contributor Author

image

image

@DeFlanko
Copy link
Contributor Author

I dont mind powershell ;)

@xnorpx
Copy link
Owner

xnorpx commented Dec 26, 2024

Will take a look tonight or tomorrow thanks for the pr!

Copy link
Owner

@xnorpx xnorpx left a comment

Choose a reason for hiding this comment

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

Mainly those 2 things other than it looks good.

install_latest_blue_onyx.ps1 Outdated Show resolved Hide resolved
install_latest_blue_onyx.ps1 Outdated Show resolved Hide resolved
Added Logic to compare the installed version to the JSON on github to determine if installing a newer version is needed or not. 

Cleaned up step formatting.

Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
Added Popup box for the end user to select where to install the PATH variable. If Self Only then the Variable gets added to User if System Wide then the Variable gets added to the System. 



Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
If Multiple GPU's detected then present popup selection box, else default value to 0



Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
@DeFlanko DeFlanko requested a review from xnorpx December 27, 2024 23:26
Copy link
Owner

@xnorpx xnorpx left a comment

Choose a reason for hiding this comment

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

added comment to ensure the gpus is array

added GPUS to an Array logic

Signed-off-by: DeFlanko <[email protected]>
Update install_latest_blue_onyx.ps1
Copy link
Owner

@xnorpx xnorpx left a comment

Choose a reason for hiding this comment

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

lgtm

@xnorpx xnorpx merged commit 4f60fe5 into xnorpx:main Dec 28, 2024
6 checks passed
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