Closed cthoyt closed 1 year ago
You definitely shouldn't do 1 (use internal ETL methods), 2 is better as it uses methods that are exposed and we can provide guarantees of stability.
as all prefix and namespace synonyms are thrown away.
not seeing the problem here...? I think we may still need to mind meld a bit on the purpose of this library.
The synonyms are useless to the intended consumers of prefixmaps. We deliberately avoid Postel's principle here. You should always use the canonical prefix and canonical namespace, not an alias, when working in any kind of linked data context.
The aliases in the csvs are just these for internal maintainers to more easily view contexts. We could move this out to a separate reporting file, but the csvs are not intended to be seen by anyone other than internal maintainers.
The synonyms are useful for the purpose of prefix normalization, especially in cases where the source is no longer updated so we don't have anyone to ask to correct their IDs
Closed by #25
Right now, there's a pretty substantial loss of content in the examples like
as all prefix and namespace synonyms are thrown away. I've implemented a more fully-featured function for converting a
prefixcommons.Context
object into acuries.Converter
in https://github.com/cthoyt/curies/pull/22. The question is: should this code live inprefixmaps
orcuries
?