-
Notifications
You must be signed in to change notification settings - Fork 7
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
TG2-VALIDATION_GEOGRAPHY_STANDARD #139
Comments
The discussion in #118 suggests the need for this validation. Added here for discussion. |
@chicoreus - does this mean that all higher geography terms need to be unambiguously resolvable. What about if you only have down to country and nothing lower? |
@chicoreus question has not been answered. Is it COMPLIANT if some can be unambiguously resolved - or does all have to be unambiguously resolvable |
I feel strongly that this test must be that the entire higher geography combination is unambiguous, as it is stated. It is not really functional to do a piece-wise validation of the geographic terms and posit compliance. If you want to go the route of checking to, for example, country, I believe this test would have to change to add another parameter to say how many levels down you want to be checked. Even so, the test would not check continent, then country, then stateProvince independently - it is only the combination that makes sense to be compliant. |
@tucotuco - Are you are saying that the test would return NOT_COMPLIANT if the not EMPTY terms among dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county and dwc:municipality resulted in ambiguity? If so, it raises an interesting point that the ambiguity needs to be reported? |
The test isn't about ambiguity, it is about being non-standard against a given authority. This is a NOTSTANDARD test, not a INCONSISTENT test. In NOTSTANDARD tests we don't say why it is not standard. Indeed, we don't even make that the job of consistency tests (e.g., #67) except to test for something very specific (e.g., #62). |
@tucotuco ok. tare |
I just noticed that the wording of this test is virtually identical to #95 - they appear duplicates! #139 #95 |
I was going to use "administrative geographic terms" in the Description of one of these tests and decided to just be explicit, as we have opted for elsewhere. |
But are the tests the same, and can one be deleted? |
I think you are right. They are close enough. |
#139: COMPLIANT if the combination of [values] of dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality can be unambiguously resolved from the bdq:sourceAuthority; #95: COMPLIANT if the combination of values of dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality can be unambiguously resolved by the bdq:sourceAuthority So....there are equal in intent. I suggest we CLOSE #95 and I'll edit this one to add "values of". Votes please? |
Thanks @tucotuco. Your call. I guess the questions are then
|
The examples seem informative. #95 dwc:stateProvince="WA" Is standard, but ambiguous, other terms would be needed to disambiguate which WA. #139 dwc:continent="Oceania", dwc:country="Australia", dwc:stateProvince="Virginia" is not standard, there is no Virginia within Oceania:Australia, no such higher geography exists, while at least two WA higher geographies exist. The whiteboard diagram from Ganesville on #95 seems to suggest that #95 was being thought of as encompasing both of these at that time. |
… On Tue, Mar 29, 2022 at 5:55 PM Paul J. Morris ***@***.***> wrote:
The examples seem informative. #95 <#95>
dwc:stateProvince="WA" Is standard, but ambiguous, other terms would be
needed to disambiguate which WA. #139
<#139> dwc:continent="Oceania",
dwc:country="Australia", dwc:stateProvince="Virginia" is not standard,
there is no Virginia within Oceania:Australia, no such higher geography
exists, while at least two WA higher geographies exist.
The whiteboard diagram from Ganesville on #95
<#95> seems to suggest that #95
<#95> was being thought of as
encompasing both of these at that time.
—
Reply to this email directly, view it on GitHub
<#139 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADQ72YT7P3RHQ3W2SJAAXDVCNU3TANCNFSM4EPT7YJQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Is there a way we can reword one of them to cover both cases. It is obvious that over time the two tests that were distinct at one time have converged over time. As @chicoreus suggests, the examples are helpful. |
I think they are very different and an attempt should not be made to merge them. The fact that the Expected Responses look the same is an accident. I think the correct Expected Response for this step should be something more akin to: EXTERNAL_PREREQUISITES_NOT_MET if the bdq:sourceAuthority is not available; INTERNAL_PREREQUISITES_NOT_MET if all of the terms dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality are EMPTY; COMPLIANT if each of the NOT_EMPTY values of dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality is a STANDARD value for that term and is consistent with all of the other values in the listed fields according to the bdq:sourceAuthority; otherwise NOT_COMPLIANT There are problems inherent in what standard means here that make me think this test is not as useful as #95. I think I am convincing myself to drop this one. |
I remain fine with omitting #139. |
The thumbs up suggests that #139 does not add significantly to #95 but for the deletion of a test, I'd value @chicoreus's 'vote'. |
Because of the confusion between this test and #95, I suggest rewording as INTERNAL_PREREQUISITES_NOT_MET if all of the terms dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality are EMPTY; COMPLIANT if the values of dwc:continent, dwc:country, dwc:countryCode, dwc:stateProvince, dwc:county, dwc:municipality can be individually resolved from the bdq:sourceAuthority; otherwise NOT_COMPLIANT |
There has been discussion between Issues #95 and #139 - the wording converged over time such as the two tests appeared to be testing for the same thing. Discussion on ZOOM has resulted in separating the two. #139 is testing individual terms for validity at that level - it looks at only one level in the hierarchy at a time and checks the validity of what is there at the level. Hence the suggested change in wording above. |
The zoom discussion with @ArthurChapman, @tucotuco and @chicoreus today concluded that tests #95, #139 and #118 were going to be very difficult to implement properly given the lack of a consistent geographic terms hierarchy by comparison with the taxonomic terms. Note the issues arising from the table above for example. We will therefore remove these tests from CORE. In their place, we will
|
Splitting Information Elements into "Information Elements ActedUpon" and "Information Elements Consulted". Changed "Field" to "TestField", "Output Type" to "TestType", deleted "Warning Type" and added TestField "Specification Last Updated" |
Specifications updated to align with the current template |
The text was updated successfully, but these errors were encountered: