Skip to content

Rework static analysis docs #7668

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
Merged
merged 6 commits into from
Jul 29, 2025
Merged

Rework static analysis docs #7668

merged 6 commits into from
Jul 29, 2025

Conversation

joellabes
Copy link
Contributor

@joellabes joellabes commented Jul 21, 2025 •
edited
Loading

What are you changing in this pull request and why?

Resolves #7396:

  • Adds new visualisations to explain JIT and AOT rendering and static analysis (and the manim code used to produce them)
  • Rewrites the docs to explain the implications of unsafe analysis, and to reflect the fact that introspective models will be unsafe by default (instead of off by default).

Checklist

Preview

/docs/fusion/new-concepts

@joellabes joellabes requested a review from a team as a code owner July 21, 2025 23:13
@vercel Vercel
Copy link

vercel bot commented Jul 21, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Jul 29, 2025 10:39pm

@github-actions github-actions bot added content Improvements or additions to content size: large This change will more than a week to address and might require more than one person labels Jul 21, 2025
runleonarun
runleonarun approved these changes Jul 29, 2025
View reviewed changes
Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

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

@joellabes Approved with a couple of comments. Ignore my comment about changing the images. I realized it doesn't work.

@joellabes @runleonarun
clarify it's a SQL query
31273b5 
Co-authored-by: Leona B. Campbell <[email protected]>
@joellabes joellabes enabled auto-merge (squash) July 29, 2025 22:38
@joellabes joellabes merged commit e445f0e into current Jul 29, 2025
7 of 8 checks passed
@joellabes joellabes deleted the joellabes-revise-static-analysis-docs branch July 29, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@runleonarun runleonarun runleonarun approved these changes

Assignees
No one assigned
Labels
content Improvements or additions to content size: large This change will more than a week to address and might require more than one person
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Take another pass over the unsafe static analysis config
2 participants
@joellabes @runleonarun