chore: clean package static-analysis findings#611
Conversation
0a1a39a to
db171a1
Compare
|
Rebased this onto current The requests from #606 remain addressed: the DRDID grid omissions are documented, the deferred-import note is preserved, and the Unreleased changelog entry is included. This is ready for review from my side. I do not have permission to apply |
|
Thanks for the rebase - and apologies for the slow feedback loop on my end. I re-validated the rebased branch against current Two changes before this can go in:
Everything else looks ready. Once you've pushed those revisions I'll add the |
|
Addressed both requested revisions in
Validation after the revisions: package-wide selected Ruff rules pass, Black and Python 3.9 compilation pass, |
Closes #606.
Summary
ResolvedSurveyDesignforward reference inbootstrap_utils.pyunderTYPE_CHECKINGpower.py# noqa: WPS433marker fromspillover.pywhile preserving the deferred-import notestaggered.pyScope
This intentionally does not touch:
F401E402I001honest_did.pymathematicallnames /E741Validation
Rebased onto current
mainat7f8dfd10and reran:python -m black --check diff_diff/bootstrap_utils.py diff_diff/chaisemartin_dhaultfoeuille.py diff_diff/honest_did.py diff_diff/imputation.py diff_diff/power.py diff_diff/prep_dgp.py diff_diff/spillover.py diff_diff/staggered.pypython -m ruff check diff_diff --select F821,F541,F841,W --output-format=concisepython3.9 -m py_compile diff_diff/bootstrap_utils.py diff_diff/power.py diff_diff/spillover.py diff_diff/chaisemartin_dhaultfoeuille.py diff_diff/honest_did.py diff_diff/imputation.py diff_diff/prep_dgp.py diff_diff/staggered.pygit diff --checkPYTHONPATH=. DIFF_DIFF_BACKEND=python pytest tests/test_bootstrap_utils.py tests/test_bootstrap_chunking.py -q(94 passed)PYTHONPATH=. DIFF_DIFF_BACKEND=python pytest tests/test_power.py tests/test_methodology_power.py tests/test_spillover.py tests/test_honest_did.py tests/test_methodology_honest_did.py tests/test_imputation.py tests/test_methodology_imputation.py tests/test_prep_dgp_reversible.py tests/test_staggered_rc.py tests/test_staggered.py tests/test_chaisemartin_dhaultfoeuille.py tests/test_dcdh_heterogeneity_cell_period_coverage.py -q(1443 passed, 104 deselected)