Possible unnecessary error checking loading from included contexts

[ptgolden]

While investigating monarch-initiative/monarch-app#819, I noticed a bottleneck in loading the "merged" context in these lines in curie:

https://github.com/biopragmatics/curies/blob/47d1f22e3aba42ea365cd7e99403eacfede4af5a/src/curies/api.py#L527-L533

Is it necessary to check for duplicate prefixes on known-good prefix maps (i.e. those bundled with this package)? If not, the converter loaded here:

prefixmaps/src/prefixmaps/io/parser.py

Lines 30 to 31 in c61e812

	if isinstance(names, str):
	return load_context(names, refresh=refresh).as_converter()

can be initialized with strict=False, resulting in a speedup of about ~15s on the "merged" context. I am happy to create a PR if that sounds okay.

[ptgolden]

To be clear, strict=False would be added as a keyword to the .from_extended_prefix_map method here:

prefixmaps/src/prefixmaps/datamodel/context.py

Lines 379 to 386 in c61e812

	def as_converter(self) -> curies.Converter:
	"""
	Get a converter from this prefix map.
	:return:
	"""
	extended_prefix_map = self.as_extended_prefix_map()
	return curies.Converter.from_extended_prefix_map(extended_prefix_map)