Skip to content

Comments

Add startup scripts for server and client on macOS/Linux and Windows#728

Merged
d1donlydfink merged 7 commits intocognizant-ai-lab:mainfrom
makkzone:main
Feb 20, 2026
Merged

Add startup scripts for server and client on macOS/Linux and Windows#728
d1donlydfink merged 7 commits intocognizant-ai-lab:mainfrom
makkzone:main

Conversation

@makkzone
Copy link
Contributor

Added the startup scripts which acts as single script to run the application.

@makkzone
Copy link
Contributor Author

@d1donlydfink @Noravee please review

@makkzone
Copy link
Contributor Author

@vince-leaf please help with review

Copy link
Collaborator

@d1donlydfink d1donlydfink left a comment

Choose a reason for hiding this comment

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

Thanks @makkzone for the additions!

Please see PR for potential improvements.

In general we need to be sure that the help given works for everyone, not just you - we don't want to generate more questions than answers.

Also worth noting that our MO is one of Assigning reviewers we want to look at the PR - not just mention them in the PR text.

Copy link
Contributor Author

@makkzone makkzone left a comment

Choose a reason for hiding this comment

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

@d1donlydfink @vince-leaf Thank you for reviewing it.

I have made the changes accordingly.

I have seen some references to Neuro-SAN in existing code, I have updated them too.

Copy link
Collaborator

@d1donlydfink d1donlydfink left a comment

Choose a reason for hiding this comment

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

Nice!
Thanks for the effort!

Copy link
Contributor

@vince-leaf vince-leaf left a comment

Choose a reason for hiding this comment

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

LGTM

@d1donlydfink d1donlydfink merged commit 196e70a into cognizant-ai-lab:main Feb 20, 2026
2 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.

3 participants