-
Notifications
You must be signed in to change notification settings - Fork 394
AB for high leverage #1816
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
AB for high leverage #1816
Conversation
✅ Deploy Preview for vigilant-albattani-df38ec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Deploying gmx-interface with
|
Latest commit: |
8666bb2
|
Status: | ✅ Deploy successful! |
Preview URL: | https://1170270f.gmx-interface.pages.dev |
Branch Preview URL: | https://ab-high-leverage-warning.gmx-interface.pages.dev |
77a992a
to
1d39861
Compare
3baaf84
to
e61e8d7
Compare
src/components/Synthetics/TradeBox/hooks/useShowHighLeverageWarning.ts
Outdated
Show resolved
Hide resolved
[BOTANIX]: ["BTC"], | ||
}; | ||
|
||
const MAX_MAJOR_TOKEN_LEVERAGE = 15n * BASIS_POINTS_DIVISOR_BIGINT; |
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.
maybe let's move to config?
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.
moved to AB config
src/config/ab.ts
Outdated
// Config for deterministic ab flags based on address | ||
|
||
export const AB_HIGH_LEVERAGE_WARNING_GROUP = "alert-high-leverage"; | ||
export const AB_HIGH_LEVERAGE_WARNING_PROBABILITY = 50n; |
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.
maybe make it a normal number like 0.5?
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.
ok
const prefferedProbabilities = [1, 10, 20, 30, 40, 50, 60, 70, 80, 90, 100]; | ||
for (const probability of prefferedProbabilities) { | ||
const count = 50_000; | ||
let yesses = 0; |
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.
please create a normal name
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.
ok
6d7b892
to
029f5d3
Compare
…-leverage-warning
…high-leverage-warning
Uh oh!
There was an error while loading. Please reload this page.