-
Notifications
You must be signed in to change notification settings - Fork 53
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
Data NO #424
Data NO #424
Conversation
I have updated your branch, so it now does not show warnings on But you still have validation errors: https://github.com/clarin-eric/ParlaMint/actions/runs/3384809580/jobs/5622227303#step:4:282
|
Thanks for your comments @matyaskopp Unfortunatly, I am am confused by some of the validation errors.
|
I think it means (but @matyaskopp pls. correct me if I am wrong) that they are a member of the same organisation twice at some time.
No, this is explained in the Guidelines - we require parliamentaryGroups, and politicalParties are optional. A "hack" is that you just rename |
yes. the awk '{gsub(/(<[a-zA-Z:]+)/,"& LINE=\"" NR "\"",$0);print}' Data/ParlaMint-NO/ParlaMint-NO.xml \
| java -jar /usr/share/java/saxon.jar -xsl:Scripts/validate-parlamint-particDesc.xsl -s:- 2>&1 \
| grep ERROR output:
ignore these lines (it is detection of one day overlap, but the message is wrong):
should be
|
Thanks for the help guys! I found the error - when a person went directly from one ministerial post to another they got two memberships with overlapping dates. I now tried merging the government memberships into one affiliation. But when doing this I got some new errors. I wonder what you guys think about this case: Now the validator gives an error on current ministers that have changed jobs. I.e. they have two "minister" affiliation, one with a "to" date and one without, but only one "member" to government. This membership does not have a "to" attribute as it is current. In these cases I now get an |
An example:
|
this affiliation does not have <affiliation role="minister" from="2021-10-14" ref="#government.NO">
<roleName xml:lang="nob">Norges fiskeriminister</roleName>
</affiliation> so from |
Does he still need a to attribute if he is still in the job? |
if he is still in job, then he is still <affiliation from="2021-10-14" to="2022-03-07" role="member" ref="#government.NO"/> |
Or leave out the |
Ah my fault - at first gave a wrong example, and had to update my comment. |
Sorry about the confusion guys |
This is the post I am confused about:
|
this should work, I have no error for this: <person xml:id="person.TEST">
<persName>
<forename>TEST</forename>
<surname>TEST</surname>
</persName>
<affiliation role="minister" from="2021-10-14" to="2022-04-12" ref="#government.NO">
<roleName xml:lang="nob">Norges kommunal- og regionalminister</roleName>
</affiliation>
<affiliation role="minister" from="2022-04-12" ref="#government.NO">
<roleName xml:lang="nob">Norges forsvarsminister</roleName>
</affiliation>
<affiliation from="2021-10-14" role="member" ref="#government.NO"/>
</person> |
Thanks for looking into this! Maybe I'm missing something? Here is a an entry and the corresponding error message:
|
I am not able to reproduce this error. Can you please update the data in your repository? I will check it then |
Sample for review. I am still getting a lot of warnings about the parliamentary replacements, but it seems they are ok?