search

ucam-department-of-psychiatry / crate

Create and use de-identified research databases. Preprocess, extract text, anonymise/de-identify, link, apply natural language processing, query for research, manage consent for contact.
GNU General Public License v3.0
19 stars 7 forks source link

Possible bug for multiprocessing.Pool #60

Closed OshinoMoumou closed 3 years ago

OshinoMoumou commented 3 years ago

Greetings! I noticed that method googleImage referred a StackOverflow post, while a recent comment to it mentioned that there is a bug in multiprocessing.Pool. I'm trying to learn the usefulness of such small updates on StackOverflow. Would this comment help improve your code? I understand that such defect might not happen in real life situation. In that case, do you think this comment can help prevent future bugs (for example, when the code were reused somewhere else)? I'll really appreciate it if you could kindly give me some feedback or suggestions. Thank you very much for your time. Have a nice day!

RudolfCardinal commented 3 years ago

Thanks! It's generally helpful to be aware of such things. However, it's not clear that this bug causes a problem in our situation, so I'll close it here unless you have found a problem empirically with our code. All the best, Rudolf