Skip to content

Add sas_data_maker_vulnerable_persons.yaml#155

Merged
mccalluc merged 6 commits intoopendp:mainfrom
mccalluc:patch-2
Feb 20, 2026
Merged

Add sas_data_maker_vulnerable_persons.yaml#155
mccalluc merged 6 commits intoopendp:mainfrom
mccalluc:patch-2

Conversation

@mccalluc
Copy link
Contributor

@mccalluc mccalluc commented Feb 3, 2026

@mccalluc
Copy link
Contributor Author

mccalluc commented Feb 3, 2026

@ganevgv : I added a TODO to double check the release_type, and a fixed a couple schema validation errors. The syntax can be confusing. Example:

    sources: |
      - Usecase by ICO: https://ico.org.uk/for-organisations/uk-gdpr-guidance-and-resources/data-sharing/privacy-enhancing-technologies/case-studies/synthetic-data-to-test-the-effectiveness-of-a-vulnerable-persons-detection-system-in-financial-services/
      - Usecase by Nationwide: https://medium.com/nationwide-technology/hazy-synthetic-data-to-fuel-rapid-innovation-fd24f2e21685
      - PrivBayes paper: https://dl.acm.org/doi/10.1145/3134428
      - DoppelGANger paper: https://arxiv.org/abs/1909.13403
    registry_authors:
      - Georgi Ganev
  • sources is a block-quoted string with the |: downstream, it is interpreted as markdown, and we get a bullet list.
  • registry_authors is required to be an array in the schema, so no |: Instead the - marks array members.

Right now, there's also a check that the slug matches the data in the record. The result can be long, but I don't think we need to revisit that design decision in this PR. Another example of a long PR is:

url_slug: korean-government-statistical-data-hub-platform-kostat-national-statistics-office-of-korea--2021

Copy link

@bhaswarpurohit bhaswarpurohit left a comment

Choose a reason for hiding this comment

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

LGTM

@mccalluc mccalluc merged commit cbf9428 into opendp:main Feb 20, 2026
1 check passed
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.

3 participants