6721 bundle syntax # as alias of ::

[wyli]

fixes #6721

Description

compatible syntax by normalising any # in ids to ::

from monai.bundle import ConfigParser

config = {
    "my_dims": 2,
    "dims_1": "$@my_dims + 1",
    "my_net": {"_target_": "BasicUNet", "spatial_dims": "@dims_1", "in_channels": 1, "out_channels": 4},
}
# in the example $@my_dims + 1 is an expression, which adds 1 to the value of @my_dims
parser = ConfigParser(config)
print(parser.get_parsed_content("my_net::spatial_dims"))  # returns 3
print(parser.get_parsed_content("my_net#spatial_dims"))  # returns 3

new test cases:

MONAI/tests/test_config_parser.py

Lines 317 to 321 in 66b50fb

	def test_slicing(self):
	config = {"test": [1, 2, 3, 4], "test1": "$@test[::]", "test2": "$@test[::-1]", "st": "aten::relu"}
	self.assertEqual(ConfigParser(config).test1, [1, 2, 3, 4])
	self.assertEqual(ConfigParser(config).test2, [4, 3, 2, 1])
	self.assertEqual(ConfigParser(config).st, "aten::relu")

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[ericspod]

This is a simple solution but it appears to work just fine, we can assume there won't be comments ever so there's no issue removing #. An alternative is to use re.split(f"#|{ID_SEP_KEY}", str(id)) or something like this but it doesn't seem necessary.

[wyli]

/integration-test

[KumoLiu]

Thanks, looks good to me.

[wyli]

/build

[wyli]

/build