Mingxin Zheng

Results 51 comments of Mingxin Zheng

The code is under refractoring, and the status switch back to WIP

The code is under refactoring, and the status switch back to WIP > Thanks for the quick update. I would suggest to split the `stats_collector.py` into several python files and...

Update: refractoring is near its end. Doc-strings / type def / units tests for the new classes are pending

@Nic-Ma I have the same goal. After the auto3dseg is merged, I will use `recursive_get_key_chains` (which outputs "AAA#BBB#CCC" lists) with other recursive get/set in ConfigParser to replace key chains. Meanwhile,...

> Hi @mingxin-zheng , > > Actually, I feel we should not expect users to use both `key chain` and `bundle config ID`. Better to unify to "AAA#BBB#CCC"? @wyli What...

@wyli Thanks for bring up the issue. What's your opinion to deprecate the `meta_key_postfix`? I think we can either - Remove the `meta_key_postfix` in the next release - Add deprecation...

@wyli @Maxime-Perret I will have access to a GPU windows system later today, and plan to update the path in the Auto3Dseg [tutorial](https://github.com/Project-MONAI/tutorials/tree/main/auto3dseg/notebooks)

@Maxime-Perret After some debugging, I would suggest replacing this line ```python config_yaml = os.path.join(config_dir, file) ``` with ```python config_yaml = Path(os.path.join(config_dir, file)).as_posix() ``` The path problem occurred when we add...

Hi @wyli , do you mean replacing single quotes with double? This is more of an issue from PythonFire rather than the subprocess call. Here is an example how to...

It is possible that file paths are not frequently used as inputs in Python Fire. For a list of 2 file strings, one needs to put up something like: `"'C:\\\\pathA\\\\fileA','C:\\\\pathB\\\\fileB'"`...