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.

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

Near: v1.signer users #7597

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Near: v1.signer users #7597

wants to merge 4 commits into from

Conversation

bh2smith
Copy link
Collaborator

@bh2smith bh2smith commented Jan 30, 2025

As far as I can tell this is the first spell for near blockchain data so the standards aren't yet well defined.

This query extracts the near accountId and derivation path used for the signing contract v1.signer. It takes too long to run and should probably become an incremental strategy (although I am seeking some pointers on how best to achieve that).

Sample PoC Query: https://dune.com/queries/4642523 (also referenced comments on SQL file).

⚠️ It takes 18 minutes to run on a large cluster and should become an incremental strategy based on block_height (since historical data is fixed).

I think that the source naming may not be defined appropriately.

Ideally the resulting source winds up being something like

or maybe:

near_v1_signer.users
v1_signer_near.users

A possible alternative project name might be

mpc_near.users.

seeking some clarification and corrections on syntax declarations (its been a while since I made a new spell cc @jeff-dude).

@github-actions github-actions bot marked this pull request as draft January 30, 2025 10:14
@github-actions github-actions bot added WIP work in progress dbt: daily covers the Daily dbt subproject labels Jan 30, 2025
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need for this source file, as it's a new model you're building
in order to read your actual source tables used in model, you'll need to introduce near source file here:
https://github.com/duneanalytics/spellbook/tree/main/sources/_base_sources/other

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Is there any particular format for these _base_sources? Or do I just move this file over there?

@jeff-dude jeff-dude self-assigned this Jan 30, 2025
bh2smith and others added 2 commits January 31, 2025 00:31
Co-authored-by: jeff-dude <102681548+jeff-dude@users.noreply.github.com>
@bh2smith
Copy link
Collaborator Author

Thanks @jeff-dude - I've addressed two. Just not sure about the _base_sources since the near domain already exists but the file isn't here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt: daily covers the Daily dbt subproject WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants