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

+ Participating entities #106

Open
wants to merge 1 commit into
base: new-arch
Choose a base branch
from
Open

+ Participating entities #106

wants to merge 1 commit into from

Conversation

prekucki
Copy link
Member

@prekucki prekucki commented Oct 1, 2024

No description provided.

@prekucki prekucki requested a review from dopiera October 1, 2024 09:19
Copy link
Collaborator

@dopiera dopiera left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 1 files reviewed, 4 unresolved discussions (waiting on @prekucki)


arch-snapshot/arch.md line 101 at r1 (raw file):

#### Yagna daemon

aka Golem Node.

nit: AKA

Code quote:

aka

arch-snapshot/arch.md line 103 at r1 (raw file):
I think we should insert a link to the layers here and say something along these lines:

Golem Node AKA Yagna daemon AKA simply Yagna is the building block of the Core Network. It is a daemon which network which every participant needs to run in order to join the Golem Network. The network of these daemons implements all the core functionalities of Golem Network, i.e. ...

Code quote:

The Golem network client component. It provides a "low-level" API for Golem network participants.
It delivers services for peer discovery, contract agreements,and the settlement and payment for services.

arch-snapshot/arch.md line 107 at r1 (raw file):

#### Provider agent

A resource-selling application. It uses the Golem API to broadcast an offer on the market and negotiate a contract. It is

Rather than talking about the Golem API, can we say it interacts with a Yagna to use the Core Network?

Also, can you please also include information about existing provider agents and what they do to give the user a chance to build an intuition?


arch-snapshot/arch.md line 113 at r1 (raw file):

A resource buying application. It uses the Golem API to find an offer on the market and negotiate a contract. It is
responsible for price negotiations and verifying costs during service agreement.

Ditto - can you please apply the same comment I left for the provider agent? Also, I think it is worth mentioning the SDKs and the golem compose to give the users an intuition of what they have ready made and what is prepared for them.

Code quote:

A resource buying application. It uses the Golem API to find an offer on the market and negotiate a contract. It is
responsible for price negotiations and verifying costs during service agreement.

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