-
Notifications
You must be signed in to change notification settings - Fork 6
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
Empty author organization name #3
Comments
Otherwise it uselessly shows up as N/A. Works around data source issue https://github.com/ietf-ribose/relaton-data-misc/issues/3
This is a data error. We have to update this entry manually since it is a static entry. |
And in this case the publisher isn’t IETF, it’s IEEE. |
Unless we are to distinguish the publisher of the bibliographic item entry vs the publisher of the information resource… but that is not supported by Relaton. |
Yeah, I think this is a useful distinction to make. There are three levels of metadata that we could be dealing with, of which Relaton covers two:
If we wanted to cover “bibliographic item publisher”, it would come at level 2, but we currently have no properties of the sort. As a side note, I think Relaton structure could do a better job at separating levels 1 and 3, e.g. by segregating them more cleanly under nested keys. Not that it’s critical… |
|
A possibly related issue—empty affiliated organization name: relaton-data-misc/data/reference.W3C.xml-c14n.yaml Lines 39 to 43 in 655752c
|
@andrew2net can you have a look? This seems to be an issue. |
The source has an empty |
@andrew2net yes we should just remove the elements that are empty. |
@ronaldtse @strogonoff fixed but the repo will be updated later |
https://github.com/ietf-ribose/relaton-data-misc/blob/1f9465d443de097c8be6dac02f1db2b74eddace0/data/reference.IEEE.802-11A.1999.yaml#L37
The text was updated successfully, but these errors were encountered: