uber-research / deep-neuroevolution

Deep Neuroevolution
Other
1.63k stars 298 forks source link

Fixed cuncurency related bug with worker_callback implementation. #36

Closed yaricom closed 4 years ago

yaricom commented 4 years ago

Return worker back to queue only after cleaning the cache to avoid possible race conditions. Otherwise the worker can be reused and come back to this method while cache become cleared from previous visit and this will result in key error.

fps7806 commented 4 years ago

Hi yaricom,

Thank you very much for your contribution! Nice catch with that very subtle bug.