Giters

sjoerdk / idis

Image DeIdentification Service, a wrapper around RSNA CTP for anonymizion of medical images

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Exception in one stream will halt processing of all streams

sjoerdk opened this issue · comments

commented

Describe the bug
Exception in one stream will halt processing of all streams

To Reproduce
Steps to reproduce the behavior:

  • Send a study to a stream, make sure something goes wrong so the folder for that study remains
  • Send the same study again (will cause exception due to folder already existing)
  • Now streams without a problem will not be processed

Expected behavior
Exceptions in one stream should be logged, but processing of other streams should continue

commented

Actually, an exception while pushing a study is a problem that should be fixed and not ignored. Exceptions should not happen regularly. If they do, the cause should be fixed so it does not happen again.
An exception in a stream can halt all other streams. The exception should just be fixed