Skip to content

Commit

Permalink
Merge pull request #525 from NCATSTranslator/NoneType_Error
Browse files Browse the repository at this point in the history
adding extra logging info
  • Loading branch information
ShervinAbd92 authored Nov 8, 2023
2 parents 8d2d0b8 + 3f7ae32 commit e0dfb56
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tr_sys/tr_ars/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -1199,11 +1199,11 @@ def merge_received(parent_pk,message_to_merge, agent_name, counter=0):
return new_merged_message
except Exception as e:
logging.exception("problem with merging for %s :" % agent_name)
raise e
#If anything goes wrong, we at least need to unlock the semaphore
#TODO make some actual proper Exception handling here.
parent.merge_semaphore=False
parent.save()
logging.info("return empty dict for merged results")
return {}
else:
#If there is currently a merge happening, we wait until it finishes to do our merge
Expand Down

0 comments on commit e0dfb56

Please sign in to comment.