-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add Plasma support to Balancer V3 models in Hourly Spellbook and DEX #9032
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
base: main
Are you sure you want to change the base?
Conversation
…ockchain :wq :wq :wq
Feat/add plasma blockchain
…defined alias error
This commit introduces new Balancer V3 pools and general DEX trading logic for the Plasma blockchain, enabling data abstraction for this chain. Key changes and dependency fixes across subprojects: - **New Models:** Introduced models in hourly_spellbook and dex for Plasma data processing. - **Dependency Resolution:** Added all necessary source definitions (e.g., balancer_v3_plasma, tokens_plasma, aave_v3_plasma, metamorpho_factory_plasma) to fix over a dozen 'Source Not Found' errors. - **Project Fixes:** Resolved the cyclic dependency in Balancer BPT models and corrected schema references in the erc4626_token_mapping.sql model (Aave to aave_v3_plasma).
This commit introduces new Balancer V3 pools and general DEX trading logic for the Plasma blockchain, enabling data abstraction for this chain. Key changes and dependency fixes across subprojects: - **New Models:** Introduced models in hourly_spellbook and dex for Plasma data processing. - **Dependency Resolution:** Added all necessary source definitions (e.g., balancer_v3_plasma, tokens_plasma, aave_v3_plasma, metamorpho_factory_plasma) to fix over a dozen 'Source Not Found' errors. - **Project Fixes:** Resolved the cyclic dependency in Balancer BPT models and corrected schema references in the erc4626_token_mapping.sql model (Aave to aave_v3_plasma).
Removed plasma schema configuration from dbt_project.yml
Added database and schema fields for Aave V3 Plasma.
Add hyperevm on spellbook
…o resolve compilation error.
…sma' to support Balancer V3 Plasma models
…symbol and removing DISTINCT from selection
… unique combination of columns
PR SummaryIntroduces full Plasma coverage for Balancer V3 and wires it into existing cross-chain rollups.
Written by Cursor Bugbot for commit 84e9b93. Configure here. |
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.
Comment @cursor review or bugbot run to trigger another review on this PR
| meta: | ||
| blockchain: arbitrum, avalanche_c, base, ethereum, gnosis, optimism, polygon, zkevmn | ||
| contributors: victorstefenon, thetroyharris, viniabussafi | ||
| blockchain: arbitrum, avalanche_c, base, ethereum, gnosis, optimism, polygon, zkevmn, plasma |
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.
Plasma models missing from aggregation union queries
High Severity
The PR creates Plasma-specific models and updates schema files to document Plasma support in the blockchain metadata, but several aggregation SQL files were not updated to include the new Plasma models in their union queries. The affected files include balancer_bpt_supply.sql, balancer_bpt_prices.sql, balancer_transfers_bpt.sql, balancer_bpt_supply_changes.sql, balancer_bpt_supply_changes_daily.sql, balancer_token_balance_changes.sql, and balancer_token_balance_changes_daily.sql. This causes Plasma data to be absent from the main aggregated tables despite the documentation indicating it should be present.
Description: Adds full support for Plasma data across multiple Balancer V3 models on Hourly Spellbook and DEX.
Changes:
Updated models under dbt_subprojects/hourly_spellbook/models/_project/balancer:
Updated models under dbt_subprojects/dex/models/_projects/balancer:
Added Plasma schemas and updated existing Balancer V3 sources accordingly.
[...]
quick links for more information: