Skip to content

create a more robust public health example with cleaned up docs - #15

Merged
Jacobihi merged 3 commits into
mainfrom
update-docs
Dec 3, 2025
Merged

create a more robust public health example with cleaned up docs#15
Jacobihi merged 3 commits into
mainfrom
update-docs

Conversation

@Jacobihi

@Jacobihi Jacobihi commented Dec 3, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@Jacobihi
Jacobihi requested a review from lbraun December 3, 2025 15:51
lbraun
lbraun previously approved these changes Dec 3, 2025

@lbraun lbraun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great @Jacobihi ! Just a few small questions


The engine minimizes cell suppression while checking all possible dimension combinations to prevent data disclosure.

**Configuration Examples based on reporting needs:**

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
**Configuration Examples based on reporting needs:**
**Configuration examples based on reporting needs:**

"population_value": 7500,
"age_range": "18_30",
"expected_to_be_redacted": true,
"note": "Suppressed due to latent revelation: County B 70_plus was a small cell in same peer group"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This note is a little confusing to me. Why was County A 18_30 suppressed due to County B 70_plus being suppressed? Did you mean County A?

Suggested change
"note": "Suppressed due to latent revelation: County B 70_plus was a small cell in same peer group"
"note": "Suppressed due to latent revelation: County A 70_plus was a small cell in same peer group"

"population_value": 8000,
"age_range": "70_plus",
"expected_to_be_redacted": false,
"note": "Safe to report, >= 11 cases. Total county C Dec: 162 cases"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Total county C Dec: 162 cases

Not sure this is important, but why do you include this info in some cases and not others?

@Jacobihi

Jacobihi commented Dec 3, 2025

Copy link
Copy Markdown
Contributor Author

@lbraun cleaned up the copy a bit based on your feedback.

@Jacobihi
Jacobihi merged commit f075e1e into main Dec 3, 2025
1 check passed
@Jacobihi
Jacobihi deleted the update-docs branch December 3, 2025 18:47
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