AllenInstitute / npc_sessions

Tools for accessing and packaging data from behavior and epyhys sessions from the Mindscope Neuropixels team, in the cloud.
1 stars 1 forks source link

Fix lightning pose table creation #125

Closed bjhardcastle closed 1 month ago

bjhardcastle commented 2 months ago

for every session we get the following warnings - check if they're real:

2024-20-09 06:55:37 | npc_sessions.sessions | WARNING | 674562_2023-10-05 behavior predictions lightning pose face parts output has wrong shape 0, expected 423404 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
2024-20-09 06:55:43 | npc_sessions.sessions | WARNING | 674562_2023-10-05 behavior error lightning pose face parts output has wrong shape 0, expected 423404 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
2024-20-09 06:55:50 | npc_sessions.sessions | WARNING | 674562_2023-10-05 behavior temporal_norm lightning pose face parts output has wrong shape 0, expected 423404 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
2024-20-09 06:56:17 | npc_sessions.sessions | WARNING | 674562_2023-10-05 face predictions lightning pose face parts output has wrong shape 0, expected 423414 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
2024-20-09 06:56:24 | npc_sessions.sessions | WARNING | 674562_2023-10-05 face error lightning pose face parts output has wrong shape 0, expected 423414 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
2024-20-09 06:56:31 | npc_sessions.sessions | WARNING | 674562_2023-10-05 face temporal_norm lightning pose face parts output has wrong shape 0, expected 423414 frames.
Lightning pose face parts capsule was likely run with an additional data asset attached
arjunsridhar12345 commented 2 months ago

ah ha I think the check should be `len(timestamps) != len(result_df) in line 2908 in sessions.py.

The current check is true all the time!

bjhardcastle commented 2 months ago

Hehe thanks - I just found the same thing

From: arjunsridhar12345 @.> Sent: Friday, September 20, 2024 10:27 AM To: AllenInstitute/npc_sessions @.> Cc: Ben Hardcastle @.>; Assign @.> Subject: Re: [AllenInstitute/npc_sessions] Fix lightning pose table creation (Issue #125)

ah ha I think the check should be `len(timestamps) != len(result_df) in line 2908 in sessions.py.

The current check is true all the time!

- Reply to this email directly, view it on GitHubhttps://github.com/AllenInstitute/npc_sessions/issues/125#issuecomment-2364186817, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APD42FHW3PQZ6N474AJG47DZXRLGJAVCNFSM6AAAAABOSHWKQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGE4DMOBRG4. You are receiving this because you were assigned.Message ID: @.**@.>>