AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-mapper

Update FIP mapper to use passed patch_cable to fiber connection info

Đang mở
#416 1 bình luận 0 reaction 1 người được giao Được @dougollerenshaw nhận Xem trên GitHub
Ngôn ngữ chính
Python
Star
0
Fork
2
Merge trung bình
1 ngày 7 giờ
Pull request đã merge (30 ngày)
1

Mô tả

Currently the FIP mapper makes assumptions about the patch_cord to fiber connection scheme. See the detailed issue description here: https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip/issues/40.

Ultimately, it'd be preferable for this information to be specified in the extracted fiber metadata. After:
* https://github.com/AllenNeuralDynamics/Aind.Physiology.Fip/issues/40
* https://github.com/AllenNeuralDynamics/aind-metadata-extractor/issues/35

are resolved, we should update the FIP mapper to:
* Remove the hard-coded assumptions about the patch_cord to fiber mapping
* Verify that all referenced fibers exist in the procedures for the given subject
* Verify that the number of active patch cords is less than or equal to the number of implanted fibers
* Verify that no duplicate mappings exist (i.e. each patch cord maps to only one fiber)

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.