Should input/output names be precached?

[aldanor]

Why do this every time run() is called, would it be possible to do it upon load?

onnxruntime-rs/onnxruntime/src/session.rs

Lines 383 to 406 in 32e9935

	let input_names: Vec<String> = self.inputs.iter().map(|input| input.name.clone()).collect();
	let input_names_cstring: Vec<CString> = input_names
	.iter()
	.cloned()
	.map(|n| CString::new(n).unwrap())
	.collect();
	let input_names_ptr: Vec<*const i8> = input_names_cstring
	.into_iter()
	.map(|n| n.into_raw() as *const i8)
	.collect();
	let output_names: Vec<String> = self
	.outputs
	.iter()
	.map(|output| output.name.clone())
	.collect();
	let output_names_cstring: Vec<CString> = output_names
	.into_iter()
	.map(|n| CString::new(n).unwrap())
	.collect();
	let output_names_ptr: Vec<*const i8> = output_names_cstring
	.iter()
	.map(|n| n.as_ptr() as *const i8)
	.collect();

[nbigaouette]

Yes, that makes a lot of sense. That came from the original port of the C example. It should definitely be moved.