Skip to content

Tiny amend#36

Merged
remlapmot merged 2 commits intomainfrom
2026-03-devel
Mar 18, 2026
Merged

Tiny amend#36
remlapmot merged 2 commits intomainfrom
2026-03-devel

Conversation

@remlapmot
Copy link
Copy Markdown
Contributor

When a treatment-level subset lacks sufficient unique values, _fit_numerator/_fit_denominator append None to the fits list. summary() now skips None entries to avoid AttributeError.

When a treatment-level subset lacks sufficient unique values,
_fit_numerator/_fit_denominator append None to the fits list.
summary() now skips None entries to avoid AttributeError.
@remlapmot
Copy link
Copy Markdown
Contributor Author

As emailed - @ryan-odea - you seem to have been removed from the CausalInference org on GitHub - which means I can no longer request you as a reviewer. Honestly I think we need you re-added. Do you want to reply here if ok with this tiny change.

@ryan-odea
Copy link
Copy Markdown
Collaborator

As emailed - @ryan-odea - you seem to have been removed from the CausalInference org on GitHub - which means I can no longer request you as a reviewer. Honestly I think we need you re-added. Do you want to reply here if ok with this tiny change.

This looks good to me! I've emailed Miguel about my access. Hopefully added back to the repos soon.

@remlapmot
Copy link
Copy Markdown
Contributor Author

Thanks Ryan.

That is hopefully good news about the org and repos.

@remlapmot remlapmot merged commit d764e6a into main Mar 18, 2026
4 checks passed
@ryan-odea ryan-odea deleted the 2026-03-devel branch March 19, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants