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

[ENH] Add better error messages + filter out noisy redcap messages #359

Merged
merged 4 commits into from
Mar 16, 2024

Conversation

DESm1th
Copy link
Contributor

@DESm1th DESm1th commented Mar 16, 2024

This PR:

  • Adds a clearer error message when the XNAT server returns an empty file [datman/xnat.py, 01ca591]
  • Adds a message that's more useful for debugging when a zip file's intended ID is already in use by another file [bin/dm_link.py, 5a11267]
  • Prevents task files from being named with unparseable characters (which can break scripts that use the task files) [bin/dm_task_files.py, 01bd074]
  • Allows redcap record IDs matching a regex to be ignored, helping to limit noise when unneeded records may be in the same project [bin/dm_redcap_scan_completed.py, 230d249]

@DESm1th DESm1th requested a review from a team as a code owner March 16, 2024 00:42
@DESm1th DESm1th requested review from kimjetwav and slimnsour March 16, 2024 00:42
@github-actions github-actions bot added the feature Tagged PR addes a new feature label Mar 16, 2024
@DESm1th DESm1th merged commit 1a6a1e0 into TIGRLab:main Mar 16, 2024
4 of 6 checks passed
@DESm1th DESm1th deleted the update_errors branch March 16, 2024 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Tagged PR addes a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant