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

docs: Fix missing setup instructions #377

Merged
merged 7 commits into from
Dec 21, 2024

Conversation

joeyparrish
Copy link
Contributor

Nowhere in the README or the bridge data template did it mention that the worker names needed to be unique. However, using the default names led to the error "Wrong credentials to submit as this worker." I found the cause on Reddit.

This adds brief notes to both the README and the template file to try to guide people explicitly on this point. Had these notes been present, I would not have made the mistake of keeping the defaults.

@oceanmasterza
Copy link

oceanmasterza commented Dec 21, 2024 via email

joeyparrish and others added 3 commits December 21, 2024 02:44
Nowhere in the README or the bridge data template did it mention that the worker names needed to be unique.  However, using the default names led to the error "Wrong credentials to submit as this worker."  I found the cause on Reddit.

This adds brief notes to both the README and the template file to try to guide people explicitly on this point.  Had these notes been present, I would not have made the mistake of keeping the defaults.
@tazlin tazlin force-pushed the fix-missing-instructions branch from 1a45d17 to 2bce23d Compare December 21, 2024 07:44
Copy link
Member

@tazlin tazlin left a comment

Choose a reason for hiding this comment

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

I've tried to clarify the intent behind your original changes, including highlighting the dynamics in both the readme and logs as well.

@tazlin
Copy link
Member

tazlin commented Dec 21, 2024

Nowhere in the README or the bridge data template did it mention that the worker names needed to be unique. However, using the default names led to the error "Wrong credentials to submit as this worker." I found the cause on Reddit.

This adds brief notes to both the README and the template file to try to guide people explicitly on this point. Had these notes been present, I would not have made the mistake of keeping the defaults.

Thanks for taking the initiative to open an PR. Its plain to see that this is confusing behavior, so the change is welcome.

@tazlin
Copy link
Member

tazlin commented Dec 21, 2024

@oceanmasterza

Nowhere do the instructions explain that you first need python and pip either. If this is really going to take of with mainstream then an installer should take care of it.

Your situation was highly specific and we put a great deal of effort into trying to make it easier for you. However, please note that there is no requirement to already have python on machine, and that there is an installer for both windows and linux and even AMD/RoCm that does in fact handle installing everything required.

In the specific cases that you would need to install python, as mentioned in the main docs and explained here, we do explain that you need python first.

If you have any further specific problems, or if you feel the docs are still insufficient, feel free to open an issue with a more specific request.

@tazlin tazlin merged commit 39e37b5 into Haidra-Org:main Dec 21, 2024
4 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