Document Bayesian model as closed beta, fix broken #bayesian anchor - #198
Open
russell-loube-mixpanel wants to merge 2 commits into
Open
Document Bayesian model as closed beta, fix broken #bayesian anchor#198russell-loube-mixpanel wants to merge 2 commits into
russell-loube-mixpanel wants to merge 2 commits into
Conversation
The Bayesian model is in closed beta and not enabled for all projects. Adds a closed-beta callout to the Bayesian section, plus signposts at the two places a customer picks a model type (Step 5 config defaults and the Experiment Model Types reference). Also fixes three broken #bayesian anchor links. Mintlify only emits heading ids down to h4; "##### Bayesian" is an h5, so it renders with no id and every link to #bayesian was dead. Adds an explicit <span id="bayesian" /> anchor, matching the workaround already used in docs/tracking-methods/sdks/swift.mdx. Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Contributor
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| docs/experiments.mdx | Clearly scopes Bayesian model availability and restores the documented in-page navigation target. |
Reviews (2): Last reviewed commit: "Merge branch 'main' into docs/bayesian-c..." | Re-trigger Greptile
myronkaifung
approved these changes
Aug 27, 2026
|
|
||
| This example shows why larger sample sizes are crucial—with only 10 users per group, even a 10-point difference in conversion rate isn't statistically significant. | ||
|
|
||
| <span id="bayesian" /> |
Collaborator
There was a problem hiding this comment.
Perhaps using #### for Bayesian is a cleaner fix than adding a custom span
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Two things, both in
docs/experiments.mdx:#bayesiananchor links.Follow-up to #159, which documented the Bayesian model but didn't mention its availability.
Closed beta
The model isn't enabled for all projects, so the docs shouldn't read as though anyone can pick it. Three changes, one callout plus two signposts at the points where a customer is actually choosing a model:
<Note>: closed beta, not enabled for all projects, how to request access, behavior may change before GA. Uses the<Note>+ bolded-lead pattern fromdocs/mcp.mdxanddocs/session-replay.mdx.Anchor fix
How do you read statistical significance?links to#bayesian, and that link is dead. The cause is the heading, not the link:Mintlify only emits heading
ids down to h4.##### Bayesianis an h5, so it renders with noid, and every link to#bayesiangoes nowhere. Confirmed against the live page —id="how-mixpanel-calculates-results"(h4) is present;id="bayesian"andid="frequentist-and-sequential"(both h5) are absent.Fixed with an explicit
<span id="bayesian" />, matching the workaround already indocs/tracking-methods/sdks/swift.mdx:84anddocs/tracking-methods/sdks/javascript.mdx:217. One anchor repairs all three call sites.I checked all 19 in-page anchor links on this page;
#bayesianwas the only broken one.Two things worth a reviewer's attention
1. How customers request access. I wrote "reach out to your account team," the phrasing this page already uses for gated Experiments features. If the beta has a specific intake — a form, a named PM, an allowlist request — that sentence should name it instead.
2. Whether the picker is hidden or visible-but-disabled. The callout says "if you don't see it in the Experiment Model type picker," which assumes non-beta projects don't see the option. If it renders greyed out instead, that phrasing needs changing.
Follow-up not done here
The
<span>anchor fixes the symptom but leaves the h5 trap in place —Frequentist and Sequentialstill has noid, and any future link to it will break the same way. The structural fix is promoting both model subsections h5 → h4, which flattens the nesting under "How Mixpanel Calculates Results" that #159 introduced. Left alone deliberately; happy to do it if preferred.Testing
No local Mintlify build in this environment, so the MDX is unrendered — worth previewing. The anchor behavior was verified against the deployed site rather than a local build.
🤖 Generated with Claude Code