forked from pymc-labs/CausalPy
-
Notifications
You must be signed in to change notification settings - Fork 0
adding vs module #1
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
Open
NathanielF
wants to merge
1,078
commits into
main
Choose a base branch
from
vs_module
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
…to multicell geolift notebook
updates: - [github.com/astral-sh/ruff-pre-commit: v0.12.0 → v0.12.1](astral-sh/ruff-pre-commit@v0.12.0...v0.12.1)
The likelihood of all models is now 2 dimensional. This means we don't have to do conditional branching for single vs multiple treatment units. So we've been able to remove a lot of the code in PyMCModel. This has touched a number of experiment classes which are not related to synthetic control.
…config [pre-commit.ci] pre-commit autoupdate
Use mock sample in tests
…_debt_v0 Refactor PyMC time series models to use xarray API
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.6 → v0.14.7](astral-sh/ruff-pre-commit@v0.14.6...v0.14.7) - [github.com/pre-commit/mirrors-mypy: v1.18.2 → v1.19.0](pre-commit/mirrors-mypy@v1.18.2...v1.19.0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
The workflow now deletes any existing 'update-uml-diagrams' branch locally and remotely before creating a new one, ensuring a clean state. PR creation will fail gracefully if a PR already exists, preventing errors in repeated runs.
) * updates to LOCAL pytest coverage * remove dependency which is in the standard library * update project urls * add keywords and classifiers * update ruff options + the fixes to comply * Fix: Disable coverage collection for doctests and extra tests * more tests to hit coverage requirements * fix failing test - maybe caused by error in resolving merge conflicts
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Introduced new skill files for designing experiments, loading datasets, performing causal analysis, and working with marimo notebooks under .claude/skills/. Added detailed reference documentation for DiD, ITS, and Synthetic Control methods. Added .cursor/commands/ files covering CausalPy demos, estimators, extras (including PlaceboAnalysis), marimo usage, methods, and research guidance. Updated .gitignore and pyproject.toml to support these additions.
Add CausalPy skills and command documentation
…ymc-labs#592) * Initial plan * Improve load_data docstring with complete dataset list and descriptions Co-authored-by: drbenvincent <[email protected]> * Fix function signature and clarify did dataset description Co-authored-by: drbenvincent <[email protected]> * Reorder datasets in docstring to match DATASETS dictionary Co-authored-by: drbenvincent <[email protected]> * Remove See Also section from load_data docstring Co-authored-by: drbenvincent <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: drbenvincent <[email protected]>
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.8 → v0.14.9](astral-sh/ruff-pre-commit@v0.14.8...v0.14.9) - [github.com/pre-commit/mirrors-mypy: v1.19.0 → v1.19.1](pre-commit/mirrors-mypy@v1.19.0...v1.19.1) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Initial plan * Initial exploration and analysis of date formatting issue Co-authored-by: drbenvincent <[email protected]> * Implement intelligent date formatting for x-axis in plots Co-authored-by: drbenvincent <[email protected]> * Address code review feedback: extract helper function and improve tests Co-authored-by: drbenvincent <[email protected]> * Final polish: Fix module docstring Co-authored-by: drbenvincent <[email protected]> * Handle long time series: vertical labels for 8+ years, smart minor gridlines Co-authored-by: drbenvincent <[email protected]> * Fix minor tick overlap and add rotation for 3-8 year range Co-authored-by: drbenvincent <[email protected]> * Fix pre-commit issues and ensure minor gridlines visible with arviz theme Co-authored-by: drbenvincent <[email protected]> * Refactor date axis formatting to use AutoDateLocator Simplified format_date_axis to always use matplotlib's AutoDateLocator and ConciseDateFormatter for automatic date tick placement and formatting. Added a maxticks parameter for controlling the maximum number of ticks. Updated tests to reflect the new behavior and removed checks for specific locators in favor of AutoDateLocator and ConciseDateFormatter. * re-run relevant notebooks --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: drbenvincent <[email protected]> Co-authored-by: Benjamin T. Vincent <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
Signed-off-by: Nathaniel <[email protected]>
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.
A staggered commit example of potential variable selection module for prior specifications like the spike and slab and horseshoe.
📚 Documentation preview 📚: https://causalpy--1.org.readthedocs.build/en/1/