Skip to content

Additional logging of cropping process#285

Open
gac55 wants to merge 4 commits intomainfrom
gc-inspect-cutter
Open

Additional logging of cropping process#285
gac55 wants to merge 4 commits intomainfrom
gc-inspect-cutter

Conversation

@gac55
Copy link
Copy Markdown
Contributor

@gac55 gac55 commented Sep 11, 2024

Partially addresses #29 in BERTIE.

Simple intervention, basic logging on the population stats before and after cropping, as well as the number of agents and households removed. A more complex intervention may be added in due course.

Checklist

Any checks which are not relevant to the PR can be pre-checked by the PR creator.
All others should be checked by the reviewer(s).
You can add extra checklist items here if required by the PR.

  • CHANGELOG updated
  • Tests added to cover contribution
  • Documentation updated

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)
- [github.com/psf/black: 24.2.0 → 24.8.0](psf/black@24.2.0...24.8.0)
- [github.com/astral-sh/ruff-pre-commit: v0.3.0 → v0.6.3](astral-sh/ruff-pre-commit@v0.3.0...v0.6.3)
- [github.com/nbQA-dev/nbQA: 1.8.3 → 1.8.7](nbQA-dev/nbQA@1.8.3...1.8.7)
@gac55 gac55 requested a review from panostsolerid October 11, 2024 09:02
@gac55 gac55 force-pushed the gc-inspect-cutter branch from e510ea3 to 9b8259d Compare October 15, 2024 08:30
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