Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new checks related to file size, presence and congruence #22

Open
mobb opened this issue Feb 15, 2019 · 4 comments
Open

new checks related to file size, presence and congruence #22

mobb opened this issue Feb 15, 2019 · 4 comments

Comments

@mobb
Copy link
Contributor

mobb commented Feb 15, 2019

per chat with Mark and Duane, 2 new checks

  1. for entity size presence: [entity]/physical/size
    entitySizePresence - statusType = warn

  2. congruence of stated value with size found when pasta downloads the entity
    entitySizeCongruence - statusType = error

these are entered in the google spreadsheet sticky rows 96, 97

since one of them has statusType error, we will convene meeting of the ECC working group to discuss and schedule.

@mobb
Copy link
Contributor Author

mobb commented Feb 15, 2019

Examining the entity size is a method for checking that this is the correct entity. So in that sense, these will operate like the 2 integrityChecksum checks, but will not control pasta behavior

@mobb
Copy link
Contributor Author

mobb commented Feb 21, 2019

related to #10

@srearl
Copy link
Collaborator

srearl commented Feb 21, 2019

Love the checks but not so sure that this issue is all that much related to #10.

@mobb
Copy link
Contributor Author

mobb commented Feb 21, 2019

it was an entity duplication problem that prompted the suggestion of this check. however in an of itself, checking file size does nothing to control duplication, although it might help identify entity1 as the same as entity2.

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

No branches or pull requests

2 participants