Skip to content

Commit

Permalink
[rllib] Log an error instead of an info event when the local inferenc…
Browse files Browse the repository at this point in the history
…e thread crashes due to an exception in the PolicyClient (ray-project#33036)

Better debug-ability when a crash occurs in the PolicyClient when using the _LocalInferenceThread. If one was running rllib with loglevel 'warning/error/critical', one would miss an exception/crash of the reason, unless the loglevel was 'info'.

Signed-off-by: mattias <[email protected]>
Co-authored-by: Kai Fricke <[email protected]>
  • Loading branch information
MattiasDC and krfricke committed Aug 14, 2023
1 parent 684dcce commit 613b97d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rllib/env/policy_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,7 @@ def run(self):
}
)
except Exception as e:
logger.info("Error: inference worker thread died!", e)
logger.error("Error: inference worker thread died!", e)


def _auto_wrap_external(real_env_creator):
Expand Down

0 comments on commit 613b97d

Please sign in to comment.