openownership / lib-cove-bods

Check that your data complies with the Beneficial Ownership Data Standard (BODS) using our install our data review library to analyse files via your command line interface
https://datareview.openownership.org/
Other
1 stars 0 forks source link

New stat/report: pepStatusDetails.missingInfoReason #30

Closed kd-ods closed 5 years ago

kd-ods commented 5 years ago

Number of Person Statements with a pepStatusDetails.missingInfoReason value which also have a hasPepStatus value.

Would give reports like:

Person Statements which have a reason for missing PEP information but which do declare a PEP status: 2

odscjames commented 5 years ago

pepStatusDetails is an array; I'm assuming from the wording of the stat that if ANY (not ALL) of the items in the array have a missingInfoReason then that person should be counted?

kd-ods commented 5 years ago

if ANY (not ALL) of the items in the array have a missingInfoReason then that person should be counted?

Yes.

odscjames commented 5 years ago

issue30

This look ok?

kd-ods commented 5 years ago

As in #29, we should make the reporting on these stats consistent in form. Here, how about:

"... where PEP status is declared AND (erroneously) a reason for missing PEP information is given: x (y%)"

(And this reported stat should come after that in #29)

odscjames commented 5 years ago

Done

odscjames commented 5 years ago

This has now been merged to master in cove and is just waiting for a deploy.