Skip to content

Conversation

@data-cowwboy
Copy link
Contributor

@data-cowwboy data-cowwboy commented Dec 9, 2025

Description:

Use underlying token prices for aTokens where the price is not available in prices.usd. This is as per the logic we have in ethereum table.


quick links for more information:

@data-cowwboy data-cowwboy marked this pull request as draft December 9, 2025 12:56
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Dec 9, 2025
@data-cowwboy data-cowwboy force-pushed the cow_atokens_underlying_asset_pricing branch from 89c4945 to 671dbca Compare December 9, 2025 14:02
@data-cowwboy data-cowwboy marked this pull request as ready for review December 9, 2025 15:09
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Dec 9, 2025
@data-cowwboy data-cowwboy changed the title added atoken prices to be used where price usd isnt there [cow protocol]added atoken prices to be used where price usd isnt there Dec 15, 2025
@cursor
Copy link

cursor bot commented Dec 15, 2025

PR Summary

Adds fallback pricing for aTokens by using underlying token USD prices in CoW Protocol trade models across multiple chains and registers the required Aave v3 sources.

  • CoW Protocol trade models (arbitrum, avalanche_c, base, bnb, gnosis, linea, polygon):
    • Add atokens_mapping and atoken_prices CTEs sourced from aave_v3_<chain>.poolconfigurator_evt_reserveinitialized and prices.usd.
    • Join atoken_prices for buy/sell tokens and coalesce with prices.usd to populate sell_price/buy_price when direct USD prices are missing.
  • Sources:
    • Register aave_v3_<chain>.poolconfigurator_evt_reserveinitialized in source YAMLs for all affected chains.

Written by Cursor Bugbot for commit 5781239. Configure here.

@jeff-dude jeff-dude self-assigned this Dec 16, 2025
@jeff-dude jeff-dude added ready-for-merging and removed ready-for-review this PR development is complete, please review labels Dec 16, 2025
@jeff-dude jeff-dude merged commit daf4c24 into duneanalytics:main Dec 17, 2025
4 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dbt: dex covers the DEX dbt subproject ready-for-merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants