-
Notifications
You must be signed in to change notification settings - Fork 10
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
Predictoor 5X Boost #824
Predictoor 5X Boost #824
Conversation
Code Climate has analyzed commit 36d46f2 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (90% is the threshold). This pull request will bring the total coverage in the repository to 93.1% (0.1% change). View more on Code Climate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I proposed a simplifying change. Sorry for the delay.
@trizin are you able to change in time?
If urgent, we can merge this and then put my proposal in afterwards.
df_py/volume/reward_calculator.py
Outdated
rewards = perc_at_j * perc_at_ij * self.OCEAN_avail | ||
apy_bound = TARGET_WPY * ocean_locked_ij | ||
dcv_bound = DCV_OCEAN_j * perc_at_ij * multiplier | ||
|
||
# check if predictoor asset and apply boost | ||
# NOTE another asset might have the same multiplier value in the future | ||
# Find a better way to check if the asset is a predictoor asset | ||
if multiplier == PREDICTOOR_MULTIPLIER: | ||
# amount of dcv boosted | ||
boosted_dcv = min(OCEAN_boost_limit_predictoor, DCV_OCEAN_j) | ||
# amount of dcv remaining | ||
remaining_dcv = max(0, DCV_OCEAN_j - OCEAN_boost_limit_predictoor) | ||
boosted_reward_bound = boosted_dcv * multiplier * 5 # 5X BOOST | ||
remaining_reward_bound = remaining_dcv * multiplier # Remaining | ||
dcv_bound = ( | ||
boosted_reward_bound + remaining_reward_bound | ||
) * perc_at_ij | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Like mentioned above, remove all these changes.)
…anprotocol/df-py into issue822-5x-predictoor-boost
Please ignore the failing Coingecko tests, not related to this PR. Here's the issue for that: #825 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Fixes #822
Changes proposed in this PR: