8000 CAPO price feed by dmitriy-woof-software · Pull Request #972 · compound-finance/comet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

CAPO price feed #972

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

dmitriy-woof-software
Copy link
Contributor

No description provided.

}

/**
* @notice Price for the latest round
Copy link

Choose a reason for hiding this comment

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

@notice comment is incorrect

Comment on lines 150 to 163
* @notice Price for the latest round
* @return roundId Round id from the underlying price feed
* @return answer Latest price for the asset in terms of ETH
* @return startedAt Timestamp when the round was started; passed on from underlying price feed
* @return updatedAt Timestamp when the round was last updated; passed on from underlying price feed
* @return answeredInRound Round id in which the answer was computed; passed on from underlying price feed
**/
function latestRoundData() external view returns (
uint80 roundId,
int256 answer,
uint256 startedAt,
uint256 updatedAt,
uint80 answeredInRound
) {

Choose a reason for hiding this comment

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

Suggested change
* @notice Price for the latest round
* @return roundId Round id from the underlying price feed
* @return answer Latest price for the asset in terms of ETH
* @return startedAt Timestamp when the round was started; passed on from underlying price feed
* @return updatedAt Timestamp when the round was last updated; passed on from underlying price feed
* @return answeredInRound Round id in which the answer was computed; passed on from underlying price feed
**/
function latestRoundData() external view returns (
uint80 roundId,
int256 answer,
uint256 startedAt,
uint256 updatedAt,
uint80 answeredInRound
) {
* @notice Price for the latest round
* @return roundId Round id from the underlying price feed
* @return answer Latest price for the asset in terms of ETH
* @return startedAt Timestamp when the round was started; passed on from underlying price feed
* @return updatedAt Timestamp when the round was last updated; passed on from underlying price feed
* @return answeredInRound Round id in which the answer was computed; passed on from underlying price feed
*/
function latestRoundData() external view returns (
uint80 roundId,
int256 answer,
uint256 startedAt,
uint256 updatedAt,
uint80 answeredInRound
) {

Comment on lines 21 to 23
* @param snapshotRatio ratio at the time of snapshot
* @param snapshotTimestamp timestamp at the time of snapshot
* @param maxYearlyRatioGrowthPercent max yearly growth percent

Choose a reason for hiding this comment

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

Suggested change
* @param snapshotRatio ratio at the time of snapshot
* @param snapshotTimestamp timestamp at the time of snapshot
* @param maxYearlyRatioGrowthPercent max yearly growth percent
* @param snapshotRatio Ratio at the time of snapshot
* @param snapshotTimestamp Timestamp at the time of snapshot
* @param maxYearlyRatioGrowthPercent Max yearly growth percent

Comment on lines 85 to 92
* @param _manager address of the manager
* @param _baseAggregatorAddress address of the base aggregator
* @param _ratioProviderAddress address of the ratio provider
* @param _description description of the pair
* @param _ratioDecimals number of decimals for the ratio
* @param _priceFeedDecimals number of decimals for the price feed
* @param _minimumSnapshotDelay minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot parameters to set price cap

Choose a reason for hiding this comment

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

Suggested change
* @param _manager address of the manager
* @param _baseAggregatorAddress address of the base aggregator
* @param _ratioProviderAddress address of the ratio provider
* @param _description description of the pair
* @param _ratioDecimals number of decimals for the ratio
* @param _priceFeedDecimals number of decimals for the price feed
* @param _minimumSnapshotDelay minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot parameters to set price cap
* @param _manager Address of the manager
* @param _baseAggregatorAddress Address of the base aggregator
* @param _ratioProviderAddress Address of the ratio provider
* @param _description Description of the pair
* @param _ratioDecimals Number of decimals for the ratio
* @param _priceFeedDecimals Number of decimals for the price feed
* @param _minimumSnapshotDelay Minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot Parameters to set price cap


/**
* @notice Updates price cap parameters
* @param priceCapParams parameters to set price cap

Choose a reason for hiding this comment

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

Suggested change
* @param priceCapParams parameters to set price cap
* @param priceCapParams Parameters to set price cap


/**
* @notice Updates price cap parameters from recent snapshot
* @param priceCapParams parameters to set price cap

Choose a reason for hiding this comment

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

Suggested change
* @param priceCapParams parameters to set price cap
* @param priceCapParams Parameters to set price cap

Comment on lines 16 to 23
* @param _manager address of the manager
* @param _baseAggregatorAddress address of the base aggregator
* @param _ratioProviderAddr 8000 ess address of the ratio provider
* @param _description description of the pair
* @param _ratioDecimals number of decimals for the ratio
* @param _priceFeedDecimals number of decimals for the price feed
* @param _minimumSnapshotDelay minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot parameters to set price cap

Choose a reason for hiding this comment

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

Suggested change
* @param _manager address of the manager
* @param _baseAggregatorAddress address of the base aggregator
* @param _ratioProviderAddress address of the ratio provider
* @param _description description of the pair
* @param _ratioDecimals number of decimals for the ratio
* @param _priceFeedDecimals number of decimals for the price feed
* @param _minimumSnapshotDelay minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot parameters to set price cap
* @param _manager Address of the manager
* @param _baseAggregatorAddress Address of the base aggregator
* @param _ratioProviderAddress Address of the ratio provider
* @param _description Description of the pair
* @param _ratioDecimals Number of decimals for the ratio
* @param _priceFeedDecimals Number of decimals for the price feed
* @param _minimumSnapshotDelay Minimum time that should have passed from the snapshot timestamp to the current block.timestamp
* @param _priceCapSnapshot Parameters to set price cap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0