建议在exception部分增加语句

[jaried]

• I have marked all applicable categories:
  • exception-raising bug
  • RL algorithm bug
  • system worker bug
  • system utils bug
  • code design/refactor
  • documentation request
  • new feature request
• I have visited the readme and doc
• I have searched through the issue tracker and pr tracker
• I have mentioned version numbers, operating system and environment, where applicable:

  import ding, torch, sys
  print(ding.__version__, torch.__version__, sys.version, sys.platform)
v0.4.3 1.12.1+cu116 3.9.7 (default, Sep 16 2021, 16:59:28) [MSC v.1916 64 bit (AMD64)] win32

建议在exception部分增加语句，比原有的格式方便debug代码：

                print(traceback.format_exc())

我自己改的地方有：

DI-engine/ding/envs/env_manager/subprocess_env_manager.py

Lines 399 to 400 in b22589c

	except BaseException as e:
	logging.warning("subprocess reset set seed failed, ignore and continue...")

DI-engine/ding/envs/env_manager/subprocess_env_manager.py

Lines 441 to 448 in b22589c

	except BaseException as e:
	if self._retry_type == 'renew' or isinstance(e, pickle.UnpicklingError):
	self._pipe_parents[env_id].close()
	if self._subprocesses[env_id].is_alive():
	self._subprocesses[env_id].terminate()
	self._create_env_subprocess(env_id)
	exceptions.append(e)
	time.sleep(self._retry_waiting_time)

DI-engine/ding/envs/env_manager/subprocess_env_manager.py

Lines 616 to 618 in b22589c

	except Exception as e:
	# when there are some errors in env, worker_fn will send the errors to env manager
	# directly send error to another process will lose the stack trace, so we create a new Exception

DI-engine/ding/envs/env_manager/subprocess_env_manager.py

Lines 673 to 674 in b22589c

	except BaseException as e:
	env.close()

DI-engine/ding/envs/env_manager/subprocess_env_manager.py

Lines 703 to 706 in b22589c

	except BaseException as e:
	logging.debug("Sub env '{}' error when executing {}".format(str(env), cmd))
	# when there are some errors in env, worker_fn will send the errors to env manager
	# directly send error to another process will lose the stack trace, so we create a new Exception

[zjowowen]

Hi,
Thanks for your suggestions!
These updates will be added in the PR:
#534