Skip to content

added blackhole avalanche_c base trades model #8524

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

added blackhole avalanche_c base trades model #8524

wants to merge 12 commits into from

Conversation

abhijeetblitz
Copy link

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

[...]

quick links for more information:

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 23, 2025 •
edited
Loading

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@github-actions github-actions bot marked this pull request as draft July 23, 2025 11:31
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Jul 23, 2025
@abhijeetblitz
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@dune-eng
Copy link

Workflow run id 16469439425 approved.

@dune-eng
Copy link

Workflow run id 16469439554 approved.

@dune-eng
Copy link

Workflow run id 16469439429 approved.

@abhijeetblitz
Copy link
Author

recheck

github-actions bot added a commit that referenced this pull request Jul 23, 2025
@github-actions
@abhijeetblitz has signed the CLA in #8524
4f1a322
@abhijeetblitz abhijeetblitz marked this pull request as ready for review July 23, 2025 11:37
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jul 23, 2025
@dune-eng
Copy link

Workflow run id 16469567383 approved.

@jeff-dude
Copy link
Member

these are both writing 0 rows, seen in logs
https://github.com/duneanalytics/spellbook/actions/runs/16469439554/job/46554767557?pr=8524#step:12:21

why could this be?

  • are the decoded tables empty? if so, were they decoded incorrectly?
  • if data in decoded tables, do the macros used here not read the data correctly?

@jeff-dude jeff-dude self-assigned this Jul 23, 2025
@jeff-dude jeff-dude added in review Assignee is currently reviewing the PR and removed ready-for-review this PR development is complete, please review labels Jul 23, 2025
@abhijeetblitz
Copy link
Author

are the decoded tables empty? if so, were they decoded incorrectly?
All the contracts submitted are decoded and accepted correctly but data is not getting populated. We're in touch with dune team on telegram and they're looking into this. There seems to some issue on dune side specific to avalanche chain and dune engineering team is looking into it. Attaching a screenshot which shows that contracts are accepted
image

@abhijeetblitz
Add blackhole avalanche_c base trades model
228fb83
@abhijeetblitz
Fix column name in blackhole avalanche_c base trades model
c86361b
@abhijeetblitz
Fix syntax error in blackhole avalanche_c base trades model by adding…
b47bfc5 
… a missing comma
@abhijeetblitz
Add blackhole_v2 and blackhole_v3 avalanche_c base trades models with…
3af1c17 
… incremental configurations and updated contributors in schema.yml
@abhijeetblitz
Remove blackhole avalanche_c base trades model SQL file
77668eb
@abhijeetblitz
Update column name in blackhole avalanche_c base trades model from Fa…
55356d4 
…ctory_evt_PairCreated to Factory_evt_PoolCreated
@abhijeetblitz
Add blackhole_avalanche_c_base_trades_seed.csv seed file with initial…
3b5be7d 
… column structure
@abhijeetblitz
Update blackhole avalanche_c base trades models to v2 and v3 with new…
e5297ff 
… configurations and contributors in schema.yml
@abhijeetblitz
Update references in avalanche_c base trades model to include blackho…
bd21d6f 
…le_v2 and blackhole_v3
@abhijeetblitz
Update source reference in blackhole_v3_avalanche_c_base_trades model…
3681f63 
… from algebrafactory_evt_custompool to algebrafactory_evt_pool
@abhijeetblitz
Update source reference in blackhole_v3_avalanche_c_base_trades model…
4b9fa0c 
… from algebrafactory_evt_pool to algebrafactory_evt_custompool
@dune-eng
Copy link

Workflow run id 16489901024 approved.

@dune-eng
Copy link

Workflow run id 16489901023 approved.

@dune-eng
Copy link

Workflow run id 16489901302 approved.

@jeff-dude jeff-dude added blocked and removed in review Assignee is currently reviewing the PR labels Jul 24, 2025
@jeff-dude jeff-dude marked this pull request as draft July 24, 2025 10:53
@github-actions github-actions bot added the WIP work in progress label Jul 24, 2025
@jeff-dude
Copy link
Member

are the decoded tables empty? if so, were they decoded incorrectly? All the contracts submitted are decoded and accepted correctly but data is not getting populated. We're in touch with dune team on telegram and they're looking into this. There seems to some issue on dune side specific to avalanche chain and dune engineering team is looking into it. Attaching a screenshot which shows that contracts are accepted image

okay, got it. marking as blocked for now. once resolved and you rerun the CI pipeline here to rebuild tables and see rows, promote PR back to ready for review.

@dune-eng
Copy link

Workflow run id 16525887423 approved.

@dune-eng
Copy link

Workflow run id 16525887422 approved.

@dune-eng
Copy link

Workflow run id 16525887480 approved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jeff-dude jeff-dude

Labels
blocked dbt: dex covers the DEX dbt subproject WIP work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abhijeetblitz @dune-eng @jeff-dude