Redesign CausalPy method selection skill#931
Merged
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #931 +/- ##
=======================================
Coverage 95.03% 95.03%
=======================================
Files 89 89
Lines 14105 14105
Branches 854 854
=======================================
Hits 13404 13404
Misses 489 489
Partials 212 212 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Make method routing more reliable for agents by replacing flat fast routing with ordered intake, text-based routing, explicit no-fit outcomes, and focused disambiguation references. Co-authored-by: Cursor <cursoragent@cursor.com>
Make the method-selection skill easier to discover for plain-language impact questions that imply quasi-experimental causal inference. Co-authored-by: Cursor <cursoragent@cursor.com>
Route comparative interrupted time-series questions to InterruptedTimeSeries with comparison series as predictors, and clarify when to prefer synthetic control. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep CITS covered in routing without elevating it in the main skill introduction. Co-authored-by: Cursor <cursoragent@cursor.com>
Rebase onto main and tighten the skill for review: trim YAML description into a triggers reference, document staggered DiD untreated-period identification support, cross-link causal-detective after a matched route, and update the skills README blurb. Co-authored-by: Cursor <cursoragent@cursor.com>
eafc26a to
1407a3b
Compare
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.
Summary
InterruptedTimeSerieswith comparison series as predictors.mainand polished for review: trimmed YAML description intoreference/triggers.md, documented staggered DiD untreated-period identification support (Warn when staggered DiD ATTs lack untreated-period support #949), and cross-linkedcausal-detectiveafter a matched route.Why now
choosing-causalpy-methodsonmainstill uses flat Fast Routing. This PR fills the agent force-fitting gap and complements #952'scausal-detectiveskill (post-selection validation).Reviewer guide
causalpy/skills/choosing-causalpy-methods/SKILL.md.reference/decision_tree.mdfor the canonical routing algorithm.its_vs_sc_vs_did.md) andmethod_capability_matrix.md.running-causalpy-experiments/SKILL.mdoutput exceptions.Out of scope
causal-detectiveintegration beyond the Matched-route cross-link.Test plan
prek runon touched skill filesmain