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

Docs: Filecoin data #1989

Closed
1 of 2 tasks
ccerv1 opened this issue Aug 25, 2024 · 10 comments · Fixed by #2130
Closed
1 of 2 tasks

Docs: Filecoin data #1989

ccerv1 opened this issue Aug 25, 2024 · 10 comments · Fixed by #2130
Assignees
Labels
c:docs Blogs, documentation, and copy

Comments

@ccerv1
Copy link
Member

ccerv1 commented Aug 25, 2024

What is the improvement or update you wish to see?

We should highlight this new dataset on our Data Portal docs.

This requires two steps:

  • Create a dbt source model that includes each of the relevant tables. See here for an example. (You can keep the database as opensource-observer)

  • Make a PR to our docs (here is the relevant page). You should also include a logo for Filecoin or Lily in the PR.

Is there any context that might help us understand?

Filecoin data on BigQuery now live!
Docs here: https://lilium.sh/data/

select * from `lily-data.lily.block_headers` where height > 4000000 limit 10

Does the docs page already exist? Please link to it.

https://docs.opensource.observer/docs/integrate/overview/

@ccerv1 ccerv1 added the c:docs Blogs, documentation, and copy label Aug 25, 2024
@ccerv1 ccerv1 self-assigned this Aug 25, 2024
@ccerv1 ccerv1 added this to OSO Aug 25, 2024
@github-project-automation github-project-automation bot moved this to Backlog in OSO Aug 25, 2024
@ccerv1 ccerv1 moved this from Backlog to Up Next in OSO Aug 25, 2024
@ccerv1 ccerv1 added this to the [c] PLN Milestones milestone Aug 25, 2024
@ryscheng
Copy link
Member

Awesome! We should also link to it in our data overview page.

Quick question --- is this dataset in the right region (US multi-region)?
If not, we may need to replicate it, like we did for Farcaster data. I can show you how we did that.

@davidgasquez
Copy link
Contributor

Dataset is in us-east4.

image

@ryscheng
Copy link
Member

Ah, in that case we'll probably need to replicate it to the US multi-region.
Would you be open to moving it there? Our pipelines require that last I checked.

If we need to replicate, here's a simple tutorial that'll make it happen:
https://docs.opensource.observer/docs/contribute/connect-data/bigquery/replication

It's similar to how we did it for Lens data
https://github.com/opensource-observer/oso/blob/main/warehouse/oso_dagster/assets/lens.py

@davidgasquez
Copy link
Contributor

Yup! I can take care of that. will probably tackle it early next week.

@davidgasquez
Copy link
Contributor

With #2033 merged, would the next step be creating the dbt models that read from opensource-observer.filecoin-lily?

@ryscheng
Copy link
Member

ryscheng commented Sep 4, 2024

Yes!!! I think the next step is to create the dbt sources, with dbt docs so that it shows up in our reference documentation:
models.opensource.observer.

I am currently working on fixing a bug in our Dagster credentialing that is preventing Dagster from actually creating the copy job. Auth issue, but I'll work on that in parallel

@ryscheng
Copy link
Member

ryscheng commented Sep 5, 2024

Debugging credential issues here
#2073

@ryscheng
Copy link
Member

ryscheng commented Sep 6, 2024

Adding to our docs here
#2088

@ryscheng
Copy link
Member

ryscheng commented Sep 6, 2024

I think the last part is the dbt source file, which will help include it in our reference documentation.

@davidgasquez lmk if you need help here, excited to close this out :-)

@ryscheng ryscheng assigned davidgasquez and unassigned ccerv1 Sep 7, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in OSO Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:docs Blogs, documentation, and copy
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants