Skip to content
This repository has been archived by the owner on Jun 6, 2023. It is now read-only.

document reward-minting taylorSeriesExpansion #338

Merged

Conversation

davidad
Copy link
Member

@davidad davidad commented May 1, 2020

Hoping to pay down my share of the technical debt mentioned in #317, by cleaning up and adding many paragraphs of documentation to the algorithm I devised for computing f(t)=1-(1/2)^(t/6yr) to atto-level precision. Please let me know if there are further questions about what's going on here.

@davidad davidad requested review from anorth and zixuanzh May 1, 2020 02:27
Copy link
Member

@anorth anorth left a comment

Choose a reason for hiding this comment

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

Thank you!

actors/builtin/reward/reward_state.go Outdated Show resolved Hide resolved
actors/builtin/reward/reward_state.go Outdated Show resolved Hide resolved
actors/builtin/reward/reward_state.go Show resolved Hide resolved
@davidad davidad merged commit 9649969 into filecoin-project:master May 5, 2020
aarshkshah1992 pushed a commit that referenced this pull request Jun 29, 2020
* document reward-minting taylorSeriesExpansion

* Apply suggestions from code review

Co-authored-by: Alex North <445306+anorth@users.noreply.github.com>

* rename LambdaBase to LnTwo

Co-authored-by: Alex North <445306+anorth@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants