-
Notifications
You must be signed in to change notification settings - Fork 2
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
Aip 5 updates #8
Conversation
Were you able to deploy this subgraph? I'm not sure if the updated version will work. Since the new parameters are to be added much later as an upgrade, when we reindex the subgraph the rpc calls to query the lower, upper and growth parameters could fail. |
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.
Can you deploy the subgraph using your github handle?
Once It's indexed we can test it locally and then point ampleforthorg/core to it.
let exp = growth.multipliedBy(delta) | ||
exp = BigNumber.maximum(new BigNumber('100'), exp) | ||
exp = BigNumber.minimum(new BigNumber('-100'), exp) | ||
const pow = new BigNumber('2').exponentiatedBy(exp) |
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.
exp=floor(exp * 32)/32
if (exp<0)
exp=ceil(exp * 32)/32
formats sdk/src/queries/policy.ts
db5fcda
to
a813ae2
Compare
rebaseFunctionLowerPercentage
,rebaseFunctionUpperPercentage
, andrebaseFunctionGrowth