Skip to content

docs(api): document 10% overestimation in collateral/pledge APIs#12922

Merged
rjan90 merged 1 commit intomasterfrom
phi/overestimation-api
Feb 26, 2025
Merged

docs(api): document 10% overestimation in collateral/pledge APIs#12922
rjan90 merged 1 commit intomasterfrom
phi/overestimation-api

Conversation

@rjan90
Copy link
Copy Markdown
Contributor

@rjan90 rjan90 commented Feb 26, 2025

Related Issues

Closes: #12913

Proposed Changes

Add documentation to API methods that intentionally overestimate values by 10%:

  • StateDealProviderCollateralBounds (min value only)
  • StateMinerPreCommitDepositForPower
  • StateMinerInitialPledgeCollateral
  • StateMinerInitialPledgeForSector

Checklist

Before you mark the PR ready for review, please make sure that:

docs(api): document 10% overestimation in collateral/pledge APIs
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Feb 26, 2025
@rjan90 rjan90 self-assigned this Feb 26, 2025
@rjan90 rjan90 requested a review from rvagg February 26, 2025 13:54
@rjan90 rjan90 added the skip/changelog This change does not require CHANGELOG.md update label Feb 26, 2025
@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Feb 26, 2025
@rjan90 rjan90 merged commit 57d6769 into master Feb 26, 2025
@rjan90 rjan90 deleted the phi/overestimation-api branch February 26, 2025 14:27
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Feb 26, 2025
@rjan90 rjan90 moved this from 🎉 Done to ☑️ Done (Archive) in FilOz Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: ☑️ Done (Archive)

Development

Successfully merging this pull request may close these issues.

Document overestimation in API calls

2 participants