Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect Delete propagation in decoding #962

Closed
samuelbray32 opened this issue May 8, 2024 · 4 comments
Closed

Incorrect Delete propagation in decoding #962

samuelbray32 opened this issue May 8, 2024 · 4 comments
Labels
bug Something isn't working infrastructure Unix, MySQL, etc. settings/issues impacting users

Comments

@samuelbray32
Copy link
Collaborator

samuelbray32 commented May 8, 2024

Describe the bug
Running a delete in UnitWaveformFeaturesGroup is trying to delete entries in DecodingOutput.SortedSpikesDecoding (which is not dependent on this table).

Note:
The attempted deletes in DecodingOutput.SortedSpikesDecoding correspond to those matching the deletion key restriction

To Reproduce

from spyglass.decoding.v1.clusterless import UnitWaveformFeaturesGroup

key = {'nwb_file_name': 'Totoro20220607_.nwb',
 'waveform_features_group_name': '04_lineartrack'}
(UnitWaveformFeaturesGroup() & key).delete(key)

Output:

[11:24:24][INFO] Spyglass: Building merge cache for unit_waveform_features_group.
Found 1 downstream merge tables
[2024-05-08 11:24:24,592][INFO]: Merge: Deleting 20 rows from decoding_merge.decoding_output
[2024-05-08 11:24:24,613][INFO]: Deleting 20 rows from decoding_merge.decoding_output__sorted_spikes_decoding_v1
[2024-05-08 11:24:24,623][INFO]: Deleting 20 rows from decoding_merge.decoding_output
[2024-05-08 11:24:41,145][WARNING]: Deletes cancelled
[2024-05-08 11:24:41,170][INFO]: Deleting 30 rows from decoding_clusterless_v1.unit_waveform_features_group__unit_features
[2024-05-08 11:24:41,180][INFO]: Deleting 1 rows from decoding_clusterless_v1.unit_waveform_features_group
[2024-05-08 11:24:43,562][WARNING]: Deletes cancelled

Expected behavior
No deletes from the sorted decoding merge table

@samuelbray32 samuelbray32 added bug Something isn't working decoding infrastructure Unix, MySQL, etc. settings/issues impacting users labels May 8, 2024
@samuelbray32
Copy link
Collaborator Author

Note: Denying the merge delete and accepting the second set of deletes worked fine. The entry is now gone from the database

@CBroz1
Copy link
Member

CBroz1 commented May 9, 2024

I wonder if this would look different once #949 is merged, because it changes how I propagate restrictions

@edeno
Copy link
Collaborator

edeno commented May 17, 2024

Do we know if #949 solved this?

@edeno edeno removed the decoding label May 17, 2024
@samuelbray32
Copy link
Collaborator Author

Tested and appears to be resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working infrastructure Unix, MySQL, etc. settings/issues impacting users
Projects
None yet
Development

No branches or pull requests

3 participants