Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix policy #231

Merged
merged 6 commits into from
Feb 27, 2024
Merged

Fix policy #231

merged 6 commits into from
Feb 27, 2024

Conversation

Adamantios
Copy link
Collaborator

To explain this PR, assume the following example:

  1. prediction sme request is sent, succeeds, and a bet is placed.
  2. claude fails
  3. prediction sme fails
  4. prediction sme request is sent, succeeds, and a bet is placed.

What we have:

  1. counts: 0, 0, reward rates: 0, 0
  2. counts: 0, 1, reward rates: 0, -0.01
  3. counts: 1, 1, reward rates: -0.01, -0.01
  4. counts: 1, 1, reward rates: -0.01, -0.01

What we should have, and this PR introduces:

  1. counts: 1, 0, reward rates: 0, 0
  2. counts: 1, 1, reward rates: 0, -0.01
  3. counts: 2, 1, reward rates: -0,005, -0.01
  4. counts: 3, 1, reward rates: -0,003333333, -0.01

The initialized value is already `0`.
To ELI5 this change when combined with 870dd88, assume the following example:
1. prediction sme request is sent, succeeds, and a bet is placed.
2. claude fails
3. prediction sme fails
4. prediction sme request is sent, succeeds, and a bet is placed.

What we have:

1. counts: 0, 0, reward rates: 0, 0
2. counts: 0, 1, reward rates: 0, -0.01
3. counts: 1, 1, reward rates: -0.01, -0.01
4. counts: 1, 1, reward rates: -0.01, -0.01

What we should have, and the commit introduces:

1. counts: 1, 0, reward rates: 0, 0
2. counts: 1, 1, reward rates: 0, -0.01
3. counts: 2, 1, reward rates: -0,005, -0.01
4. counts: 3, 1, reward rates: -0,003333333, -0.01
We set a small punishment when the price has not been set or a nevermined subscription is used to avoid repeatedly using the same tool.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Unrelated change, but useful as it prevents reusing the same tool when using subscriptions.

Copy link
Collaborator Author

@Adamantios Adamantios Feb 27, 2024

Choose a reason for hiding this comment

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

Unrelated change, but this assignment was unnecessary.

@Adamantios Adamantios merged commit 22e2b3a into main Feb 27, 2024
6 checks passed
@Adamantios Adamantios deleted the fix/policy branch February 27, 2024 16:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants