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

Use signalledTokens instead of signalAmount for minSignal threshold #367

Merged
merged 1 commit into from
Feb 23, 2022

Conversation

Jannis
Copy link
Contributor

@Jannis Jannis commented Feb 21, 2022

The signalledTokens field describes the amount of GRT signalled on a subgraph or subgraph deployment. signalAmount describes the amount of signal shares on a subgraph, which is based on the bonding curve and hence don't correspond 1:1 to the amount of GRT signalled.

It is therefore much more intuitive to use signalledTokens for the minSignal threshold in indexing rules.

The `signalledTokens` field describes the amount of GRT signalled on a
subgraph or subgraph deployment. `signalAmount` describes the amount of
signal shares on a subgraph, which is based on the bonding curve and
hence don't correspond 1:1 to the amount of GRT signalled.

It is therefore much more intuitive to use `signalledTokens` for the
`minSignal` threshold in indexing rules.
@Jannis Jannis self-assigned this Feb 21, 2022
@Jannis Jannis changed the title *: Use signalledTokens instead of signalAmount for minSignal threshold Use signalledTokens instead of signalAmount for minSignal threshold Feb 21, 2022
Copy link
Contributor

@hopeyen hopeyen left a comment

Choose a reason for hiding this comment

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

The change makes sense to me. re-ran jobs and agent image was able to build :)

@Jannis Jannis merged commit d579e1b into main Feb 23, 2022
@Jannis Jannis deleted the jannis/fix-min-signal-feature branch February 23, 2022 16:23
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