Skip to content

[Tools] delete physiological files #8232

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Dec 6, 2022
Merged

[Tools] delete physiological files #8232

merged 4 commits into from
Dec 6, 2022

Conversation

@laemtl
Copy link
Contributor

@laemtl laemtl commented Nov 17, 2022 •
edited
Loading

  • Script to delete all the files/db entries created by an eeg file ingestion.
    Usage: php delete_physiological_file.php [PID] confirm|tosql
    Split files not supported at the moment.

  • Move deletion tools into data_integrity/data_deletion

@laemtl laemtl requested a review from regisoc November 18, 2022 19:35
@laemtl laemtl force-pushed the tools_delete_physiological_file branch 2 times, most recently from 82173b3 to 8758b71 Compare November 18, 2022 20:34
@laemtl laemtl force-pushed the tools_delete_physiological_file branch from 8758b71 to 3da0189 Compare November 18, 2022 21:32
regisoc
regisoc reviewed Nov 23, 2022
View reviewed changes
Copy link
Contributor

@regisoc regisoc left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems it lacks a table for the physiological script: physiological_annotation_archive.

@laemtl laemtl force-pushed the tools_delete_physiological_file branch from 3150f9f to 21aa114 Compare December 2, 2022 16:16
@laemtl laemtl force-pushed the tools_delete_physiological_file branch from 8209150 to 913cf2d Compare December 2, 2022 23:32
@driusan driusan merged commit 1d4aa0d into aces:main Dec 6, 2022
@jeffersoncasimir jeffersoncasimir mentioned this pull request Jan 13, 2023
Merged
@ridz1208 ridz1208 added this to the 25.0.0 milestone Mar 6, 2023
driusan pushed a commit that referenced this pull request Mar 21, 2023
@jeffersoncasimir
[Tools] Modified deletion query order for physiological file deletion (…
a903575 
…#8322)

The order of deleted queries is changed, with a few extra necessary queries added.

Related to #8232
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@regisoc regisoc regisoc approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

25.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@laemtl @regisoc @driusan @ridz1208