AllenNeuralDynamics / aind-data-asset-indexer

MIT License
0 stars 0 forks source link

Adjust `external_links` input expectations #74

Open Sun-flow opened 2 weeks ago

Sun-flow commented 2 weeks ago

Aind Data Schema recently had changes which converted Metadata.external_links from being a List of Dicts to a Dict of Lists. This is a breaking change, which needs to be updated in the indexer.

Indexer will need to sort external links in the new format Dict[ExternalPlatform, List[str]]

helen-m-lin commented 2 weeks ago

This is where the indexer code needs to be updated for enabling the format change for future records: https://github.com/AllenNeuralDynamics/aind-data-asset-indexer/blob/f6cacb0a061d0cbab6de9fafc4bb28eb87809406/src/aind_data_asset_indexer/utils.py#L1020

jtyoung84 commented 1 week ago

We should wait until aind-data-schema v1.0 is rolled out