-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: setup ALM trades + feed Arrakis data #5764
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
Workflow run id 8633904472 approved. |
Workflow run id 8633904042 approved. |
I have read the CLA Document and I hereby sign the CLA |
Workflow run id 8633923184 approved. |
Workflow run id 8633923362 approved. |
Workflow run id 8634105480 approved. |
Workflow run id 8634105204 approved. |
Workflow run id 8642631739 approved. |
Workflow run id 8642631821 approved. |
Workflow run id 8643190908 approved. |
Workflow run id 8643191070 approved. |
Workflow run id 8643476519 approved. |
Workflow run id 8643476763 approved. |
Workflow run id 8643657415 approved. |
Workflow run id 8643657114 approved. |
Workflow run id 8643789626 approved. |
Workflow run id 8643789863 approved. |
Workflow run id 8643901964 approved. |
Workflow run id 8643902133 approved. |
Workflow run id 8644033338 approved. |
Workflow run id 8644033543 approved. |
Workflow run id 8644236622 approved. |
Workflow run id 8644236822 approved. |
Workflow run id 8644390760 approved. |
Workflow run id 8644390649 approved. |
@Hosuke @jeff-dude can you help me troubleshoot please? |
I think the problem may be caused by the source definition. |
schema = 'alm_ethereum' | ||
, alias = 'base_trades' | ||
, materialized = 'view' | ||
, unique_key = ['blockchain', 'project', 'version', 'tx_hash', 'evt_index', 'vault_address'] |
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.
Since this spell is materialized as view
now, I will remove incremental filters and unique_key in this spell.
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.
LGTM.✅
Thank you @0xrusowsky
By the way @0xrusowsky can you add a |
sure, where should i place it? and regarding the content, is it enough to explain what i did on the issue #5626? or should i expand it further and also detail the fields, how the queries are built, etc? |
You can add a |
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.
pushed a final commit to cleanup usage of base_
in table names as this doesn't fully follow that standard. we can look to use that approach as the sector spell grows.
once you push a quick |
Workflow run id 9301037767 approved. |
Workflow run id 9301038039 approved. |
@jeff-dude @Hosuke i made a readme proposal, feel free to do any changes you'd like! |
models/_sector/alm/README.md
Outdated
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.
Thank you for adding the readme.
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.
pushed one final commit on what i think was a copy/paste typo. thank you for adding!
Congrats, your important contribution to this open-source project has earned you a GitPOAP! GitPOAP: 2024 Dune Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
this PR will tackle:
it basically follows the structure of
dex.trades
for a new sectoralm
(active liquidity managers), so that for each trade, dune wizards can easily know the subset of swap volume which was served by an ALM.