Support pathlib.Path when paths are required in python module

[daniel-j-h]

Right now we only support string paths as in

rt.InferenceSession("myfile.pb")

We should support semantically accurate pathlib.Path types, too

rt.InferenceSession(Path("myfile.pb"))

and not throw an exception.

onnxruntime/onnxruntime/python/session.py

Lines 34 to 42 in f9bf546

	if isinstance(self._path_or_bytes, str):
	self._sess.load_model(self._path_or_bytes, providers)
	elif isinstance(self._path_or_bytes, bytes):
	self._sess.read_bytes(self._path_or_bytes, providers)
	elif isinstance(self._path_or_bytes, tuple):
	# to remove, hidden trick
	self._sess.load_model_no_init(self._path_or_bytes[0], providers)
	else:
	raise TypeError("Unable to load from type '{0}'".format(type(self._path_or_bytes)))

Note: not just in InferenceSession but everywhere we require a path.

[stale]

This issue has been automatically marked as stale due to inactivity and will be closed in 7 days if no further activity occurs. If further support is needed, please provide an update and/or more details.

[daniel-j-h]

Thanks bot! pathlib.Path support would still be a nice thing to have 😊

…

On July 18, 2020 6:16:10 AM UTC, "stale[bot]" ***@***.***> wrote: This issue has been automatically marked as stale due to inactivity and will be closed in 7 days if no further activity occurs. If further support is needed, please provide an update and/or more details.

[stale]

This issue has been automatically marked as stale due to inactivity and will be closed in 7 days if no further activity occurs. If further support is needed, please provide an update and/or more details.

[daniel-j-h]

Still an issue. I will not update this every other month to keep the bot happy from now on..