Skip to content
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

feat: add banana gun evm bot trades spellbook #7180

Merged
merged 27 commits into from
Nov 28, 2024
Merged

feat: add banana gun evm bot trades spellbook #7180

merged 27 commits into from
Nov 28, 2024

Conversation

whalehunting
Copy link
Contributor

@whalehunting whalehunting commented Nov 24, 2024 •
edited
Loading

This PR reimplements #6290 and aims to add the first EVM bot trades spellbook.

Description:

EVM bot trades data is growing, so it seems like a reasonable time to move them to Spellbook. I'm planning to do this for most EVM bots, so this spellbook should act as a blueprint for other bots.

Adds 3 new spellbooks:

  • banana_gun_ethereum_bot_trades
  • banana_gun_base_bot_trades
  • dex_evm_bot_trades

@whalehunting
init
1b79792
@whalehunting
fix: rename tests to data_tests
b9f7e02
@whalehunting
chore: disable test for now
d370927
@whalehunting
chore: add base spellbook, remove outcommented test
dfe8f9f
@whalehunting
feat: add blast spellbook
3aa5434
@whalehunting
fix: pick tx_hash column from botTrades table
71a6b4e
@whalehunting
chore: add bot column, change project_start_date
f8a598d
@whalehunting
chore: add real start date for blast
7da4dbe
@whalehunting
chore: move start date to original value
f88e453
@whalehunting
chore: remove miner bribes from spellbook schema
0fcfcc8
@whalehunting
chore: remove blast spellbook, since it has no data yet
92716b6
@whalehunting
fix: add incremental filters
efdf13f
@whalehunting
test: add check_bot_trades_seed for dex subproject
7fb971e
@whalehunting
fix: remove leading AND in where condition
74f4a00
@whalehunting
test: add base seed
75dd3ed
@whalehunting
fix: cast value column to decimal
0554c39
@whalehunting
test: add correct seed files
90b8b77
@whalehunting
fix: force version column to get parsed as string in seed
7e6e570
@whalehunting
Revert "fix: force version column to get parsed as string in seed"
78e0160 
This reverts commit 7e6e570.
@whalehunting
fix: force seed column as varchar
ef6a555
@whalehunting
fix: fix typo
1bc7fe5
@whalehunting whalehunting marked this pull request as ready for review November 24, 2024 12:24
@clizzard7 clizzard7 mentioned this pull request Nov 24, 2024
Merged
@0xRobin 0xRobin self-assigned this Nov 25, 2024
@0xRobin 0xRobin added WIP work in progress dbt: dex covers the DEX dbt subproject labels Nov 25, 2024
@0xRobin
Copy link
Collaborator

0xRobin commented Nov 25, 2024

@whalehunting Fixed the seed for you.
I did describe test_schema.git_dunesql_37fe81f_banana_gun_ethereum_trades_seed on Dune to see the actual schema and then added all the varbinary columns to the seed schema file. (The engine does not infer the varbinary type automatically, so they need to be specified)

@whalehunting whalehunting requested a review from 0xRobin November 25, 2024 09:32
@0xRobin 0xRobin added ready-for-merging and removed WIP work in progress labels Nov 25, 2024
@0xRobin 0xRobin merged commit bcb133e into duneanalytics:main Nov 28, 2024
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@0xRobin 0xRobin 0xRobin approved these changes

Assignees

@0xRobin 0xRobin

Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@whalehunting @0xRobin