tangos icon indicating copy to clipboard operation
tangos copied to clipboard

feat: Support multi-level paths for custom handlers

Open wx-ys opened this issue 6 months ago • 2 comments

This PR updates get_named_handler_class to correctly parse handler names with multi-level module paths, such as package.subpackage.ClassName.

This makes the handler loading mechanism more robust and allows for greater flexibility in organizing custom handler code.

For example, if the input is 'galyst.input_handler.ArepoHDFSubfindInputHandler', the module_name will now correctly be 'galyst.input_handler' and the class_name will be 'ArepoHDFSubfindInputHandler'.

Closes #272

wx-ys avatar Jun 29 '25 12:06 wx-ys

Sorry for the long delay in reviewing this. It looks fine, but is there a way to test it?

apontzen avatar Aug 20 '25 09:08 apontzen

Hi, I have added a simple test for this.

wx-ys avatar Aug 21 '25 07:08 wx-ys