search

linkml / prefixmaps

Semantic prefix map registry
https://linkml.io/prefixmaps/
Apache License 2.0
10 stars 3 forks source link

update dependencies #55

Closed glass-ships closed 7 months ago

glass-ships commented 7 months ago

bioregistry no longer requires pydantic <2

This will allow for prefixmaps to open up its pydantic specification, and thus multiple other packages such as koza, monarch-ingest, monarch-app, to update to pydantic v2

currently test fails on AttributeError: module 'bioregistry' has no attribute 'get_extended_prefix_map'

cthoyt commented 7 months ago

@glass-ships I changed this part of the Bioregistry's public interface, there's also a corresponding fix in https://github.com/linkml/prefixmaps/pull/52/files

cthoyt commented 7 months ago

@glass-ships it's also the case that bioregistry isn't a runtime requirement for prefixmaps, it's only used for regenerating the Bioregistry and merged prefix map

glass-ships commented 7 months ago

thanks @cthoyt! i noticed a comment in another PR that mentioned there was some overlap between this and others - let me know if my PR is redundant, I'd be happy to close as a duplicate

cthoyt commented 7 months ago

Thanks @glass-ships! Happy you were able to pull out this part and keep the "single responsibility principle" :)