Optimism Wallet Summary - WIP #4127

henrystats · 2023-08-19T14:57:59Z

Thank you for contributing to Spellbook!

Please refer to the top of the readme in the root of Spellbook to learn how to contribute to Spellbook on DuneSQL.

dune-eng · 2023-08-19T14:58:45Z

Workflow run id 5912057508 approved.

dune-eng · 2023-08-19T14:58:46Z

Workflow run id 5912057553 approved.

dune-eng · 2023-08-19T22:54:57Z

Workflow run id 5913926397 approved.

dune-eng · 2023-08-19T22:54:59Z

Workflow run id 5913926507 approved.

dune-eng · 2023-08-20T22:43:33Z

Workflow run id 5920002394 approved.

dune-eng · 2023-08-20T22:43:34Z

Workflow run id 5920002426 approved.

dune-eng · 2023-08-20T22:55:34Z

Workflow run id 5920042300 approved.

dune-eng · 2023-08-20T22:55:37Z

Workflow run id 5920042310 approved.

dune-eng · 2023-08-20T23:07:59Z

Workflow run id 5920099225 approved.

dune-eng · 2023-08-20T23:08:01Z

Workflow run id 5920099303 approved.

dune-eng · 2023-08-20T23:21:57Z

Workflow run id 5920169361 approved.

dune-eng · 2023-08-20T23:21:57Z

Workflow run id 5920169412 approved.

dune-eng · 2023-08-20T23:44:36Z

Workflow run id 5920254171 approved.

dune-eng · 2023-08-20T23:44:39Z

Workflow run id 5920254185 approved.

dune-eng · 2023-08-21T00:00:55Z

Workflow run id 5920304477 approved.

dune-eng · 2023-08-21T00:00:56Z

Workflow run id 5920304460 approved.

henrystats · 2023-08-21T00:17:10Z

depends on pr 4130

chuxinh · 2023-08-21T06:58:33Z

models/addresses_events/optimism/addresses_events_optimism_first_activity_legacy.sql

+}}
+
+SELECT 
+    1 as dummy 


add a new line below

chuxinh · 2023-08-21T06:59:48Z

seeds/addresses_events/_schema.yml

+        blockchain: varchar
+        address: varbinary
+        first_activity_to: varbinary
+        block_number: bigint


add a new line below

chuxinh · 2023-08-21T07:00:05Z

seeds/addresses_events/first_activity_seed.csv

@@ -0,0 +1,3 @@
+blockchain,address,first_activity_to,block_number,tx_hash
+optimism,0x77BA75A9a95b5aB756749fF5519aC40Ed4AAb486,0x5485798748221719181499103911646299086586,30689881,0x429f0e772d36451ad1e5c9b39a369eb7c1471cd16084209e33eb3d7abf28b56c
+optimism,0x1ddb27df0c0c20effb80d65ce7f870b2b083819c,0x8700daec35af8ff88c16bdf0418774cb3d7599b4,104546448,0x5c913779a545da143e0cff539d2197590b834614692b4c2ba7601a480a17d836


add a new line below

chuxinh · 2023-08-21T07:00:17Z

models/addresses_summary/optimism/addresses_summary_optimism_mainet_legacy.sql

+}}
+
+SELECT 
+    1 as dummmy


add a new line below

chuxinh · 2023-08-21T07:03:04Z

models/addresses_summary/optimism/addresses_summary_optimism_schema.yml

+      - name: gas_spent
+        description: "total gas spent by adddress on OP mainet"
+      - name: unique_dapps
+        description: "unique dapps address interacted with OP mainet"


add a new line below

chuxinh · 2023-08-21T07:47:47Z

models/addresses_summary/optimism/addresses_summary_optimism_mainet.sql

+            WHEN (COUNT(ot.hash)/(date_diff('day', min(ot.block_time), max(ot.block_time)) + 1)) >= 1 THEN 'Daily User'
+            WHEN (COUNT(ot.hash)/(date_diff('day', min(ot.block_time), max(ot.block_time)) + 1)) >= 0.142857142857 THEN 'Weekly User'
+            WHEN (COUNT(ot.hash)/(date_diff('day', min(ot.block_time), max(ot.block_time)) + 1)) >= 0.0333333333333 THEN 'Monthly User'
+            WHEN (COUNT(ot.hash)/(date_diff('day', min(ot.block_time), max(ot.block_time)) + 1)) >= 0.0027397260274 THEN 'Yearly User'


can you explain how these numbers are derived?

number of transactions / number of days active (age)

soo for a daily user they need to make at least one transaction per day

for a weekly user they need to make at least one transaction per week which 1/7 = 0.142857.. same for monthly and yearly as well.

I added + 1 because date_diff returns zero (causes an error since we're dividing by 0) if an address as only been active on one day... this is causing the label to be inaccurate when an address is only active one day so will fix that shortly.

chuxinh · 2023-08-21T07:51:10Z

models/addresses_summary/optimism/addresses_summary_optimism_mainet.sql

+    END as usage_frequency,
+    COUNT(ot.hash) as number_of_transactions,
+    COUNT(DISTINCT(cm.contract_project)) as unique_dapps 
+FROM 


Not sure if i've missed something here but is the logic saying that if it's incremental, then only the weekly active addresses are showing up in this table?

or is the logic here self updating the original table if an address is included in weekly active here?

yep the latter.. the table only updates data for an address that's included in the weekly active.

chuxinh · 2023-08-21T08:27:23Z

models/addresses_summary/optimism/addresses_summary_optimism_mainet.sql

A few other things could be helpful to add:

First contract interacted, or even like a path of contracts interacted for the first few transactions? You can you contract_mapping table for that

First function call

address recency

amount of op delegation (first delegated at block_number, block_time, amount etc). I previously created a table in Dune trying to do the address summary. Feel free to borrow some ideas from there: https://dune.com/queries/2794862

okee.. thank you... I'll resolve your comments and also add the other things later this evening.

chuxinh · 2023-08-21T08:27:41Z

models/addresses_summary/optimism/addresses_summary_optimism_schema.yml

+version: 2
+
+models:
+  - name: addresses_summary_optimism_mainet


Suggested change

- name: addresses_summary_optimism_mainet

- name: addresses_summary_op_mainnet

chuxinh · 2023-08-21T08:28:05Z

models/addresses_summary/optimism/addresses_summary_optimism_schema.yml

+      contributors: Henrystats
+    config:
+      tags: ['table', 'wallet summary', 'mainet', 'addresses_summary', 'optimism']
+    description: "Table showing summary of an address on Optimism mainet"


Suggested change

description: "Table showing summary of an address on Optimism mainet"

description: "Table showing summary of an address on OP mainnet"

dune-eng · 2023-08-23T15:40:46Z

Workflow run id 5953457024 approved.

dune-eng · 2023-08-23T15:41:44Z

Workflow run id 5953450960 approved.

dune-eng · 2023-08-23T15:48:50Z

Workflow run id 5953560825 approved.

dune-eng · 2023-08-23T15:48:55Z

Workflow run id 5953560712 approved.

henrystats · 2023-08-23T16:16:41Z

@chuxinh I've made the changes as requested... for the first_contract_project did you mean I should add that in first_activity as well?

Will do that in a bit if that's the case, originally left it out since I assumed you suggested that so we'd not need to do a left join with the contract mapping table but we need to because of the unique dapps column..

ci is failing now cause a couple of heavy pr's recently got merged.

dune-eng · 2023-08-24T01:47:47Z

Workflow run id 5958481062 approved.

dune-eng · 2023-08-24T01:47:48Z

Workflow run id 5958481058 approved.

chuxinh · 2023-08-24T03:54:40Z

Oh I was referring to the function signature since you only use this for the first tx

henrystats · 2023-08-24T11:57:04Z

ohh oke i had updated that @chuxinh

jeff-dude · 2023-08-24T19:25:45Z

updated to wip -- feel free to edit title & tag when ready for final review

henrystats · 2023-08-24T20:43:53Z

thank you ser!

chuxinh

Logic looks good to me! Just a few more comments and thanks so much for this!! @henrystats

chuxinh · 2023-08-25T00:16:19Z

models/addresses_summary/addresses_summary_schema.yml

+      - name: address_age
+        description: "Time period the address has been active for labelled" 
+      - name: address_age
+        description: "Time period since the address was last active for labelled" 


chuxinh · 2023-08-25T00:16:33Z

models/addresses_summary/addresses_summary_schema.yml

+        description: "Time period since the address was last active for labelled" 
+      - name: address_age_in_days
+        description: "Time period in days the address has been active for" 
+      - name: recency_age_in_days


Suggested change

- name: recency_age_in_days

- name: recency_in_days

maybe change to the name above. also remember to update that in sql

chuxinh · 2023-08-25T00:17:46Z

models/addresses_summary/addresses_summary_schema.yml

+      - name: usage_frequency
+        description: "How often address is active"
+      - name: gas_spent
+        description: "Total gas spent by an address on OP Mainnet"


Suggested change

description: "Total gas spent by an address on OP Mainnet"

description: "Total gas spent in ETH by an address on OP Mainnet"

chuxinh · 2023-08-25T00:29:08Z

models/addresses_events/optimism/addresses_events_optimism_schema.yml

+        tests:
+          - unique
+      - name: first_activity_to
+        description: "Which Optimism address received the first activity of the address with ETH"


First transaction activity to address

chuxinh · 2023-08-25T00:43:14Z

models/addresses_summary/addresses_summary_op_mainnet.sql

+            case when gas_price = cast(0 as uint256) then 0
+            else cast(gas_used as double) * cast(gas_price as double)/1e18 + cast(l1_fee as double) /1e18
+            end as gas_spent,
+            data, 


remove data column if it's not used

chuxinh · 2023-08-25T00:46:42Z

models/addresses_summary/addresses_summary_op_mainnet.sql

+    MIN_BY(cm.contract_project, ot.block_number) as first_to_project, 
+    MIN_BY(ot.to, ot.block_number) as first_to_address


maybe can also at the last to address and project?

dune-eng · 2023-08-25T01:03:56Z

Workflow run id 5970594069 approved.

dune-eng · 2023-08-25T01:03:56Z

Workflow run id 5970594186 approved.

dune-eng · 2023-08-25T01:04:04Z

Workflow run id 5970595022 approved.

dune-eng · 2023-08-25T01:04:05Z

Workflow run id 5970595097 approved.

henrystats · 2023-08-25T01:14:00Z

made the changes @chuxinh

wallet summary wip

dd45511

create addresses summary model

ba496a4

add age, frequency and gas spent

10a5682

add unique dapps

b759138

fix usage frequency

5d63a76

Test fix

79c91a6

fix join

4642353

fix seed

ef09595

henrystats changed the title ~~Optimism Wallet Summary - WIP~~ Optimism Wallet Summary - Ready for review Aug 21, 2023

chuxinh reviewed Aug 21, 2023

View reviewed changes

henrystats changed the title ~~Optimism Wallet Summary - Ready for review~~ Optimism Wallet Summary - WIP Aug 22, 2023

henrystats added 2 commits August 23, 2023 03:03

rewrite first activity

000fe28

smol test

c138c5d

Another synthax fix

9e3f969

Merge branch 'main' into first-activity

ee5ac98

jeff-dude added the WIP work in progress label Aug 24, 2023

chuxinh suggested changes Aug 25, 2023

View reviewed changes

henrystats added 2 commits August 25, 2023 02:03

make changes

ebb9e0c

Merge branch 'main' into first-activity

21af010

jeff-dude self-assigned this Aug 25, 2023

jeff-dude added in review Assignee is currently reviewing the PR and removed WIP work in progress in review Assignee is currently reviewing the PR labels Aug 25, 2023

jeff-dude approved these changes Aug 25, 2023

View reviewed changes

jeff-dude merged commit 037638b into duneanalytics:main Aug 25, 2023

github-actions bot locked and limited conversation to collaborators Aug 25, 2023

henrystats deleted the first-activity branch September 4, 2023 15:46

	- name: addresses_summary_optimism_mainet
	- name: addresses_summary_op_mainnet

	description: "Table showing summary of an address on Optimism mainet"
	description: "Table showing summary of an address on OP mainnet"

	description: "Total gas spent by an address on OP Mainnet"
	description: "Total gas spent in ETH by an address on OP Mainnet"

		MIN_BY(cm.contract_project, ot.block_number) as first_to_project,
		MIN_BY(ot.to, ot.block_number) as first_to_address

Optimism Wallet Summary - WIP #4127

Optimism Wallet Summary - WIP #4127

Conversation

henrystats commented Aug 19, 2023

Thank you for contributing to Spellbook!

dune-eng commented Aug 19, 2023

dune-eng commented Aug 19, 2023

dune-eng commented Aug 19, 2023

dune-eng commented Aug 19, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 20, 2023

dune-eng commented Aug 21, 2023

dune-eng commented Aug 21, 2023

henrystats commented Aug 21, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dune-eng commented Aug 23, 2023

dune-eng commented Aug 23, 2023

dune-eng commented Aug 23, 2023

dune-eng commented Aug 23, 2023

henrystats commented Aug 23, 2023

dune-eng commented Aug 24, 2023

dune-eng commented Aug 24, 2023

chuxinh commented Aug 24, 2023

henrystats commented Aug 24, 2023

jeff-dude commented Aug 24, 2023

henrystats commented Aug 24, 2023

chuxinh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dune-eng commented Aug 25, 2023

dune-eng commented Aug 25, 2023

dune-eng commented Aug 25, 2023

dune-eng commented Aug 25, 2023

henrystats commented Aug 25, 2023

henrystats commented Aug 21, 2023 •

edited

Loading