Added option for SimpleInferer

[vikashg]

Added an option for SimpleInferer on monai_seg_inference_operator.py

[vikashg]

@MMelQin Including StrEnum gives me an error for testing. If you can have a look that will be really helpful. Maybe I am using it wrong.

[MMelQin]

@MMelQin Including StrEnum gives me an error for testing. If you can have a look that will be really helpful. Maybe I am using it wrong.

Yes, needs to define the class of type StrEnum before using it. Will suggest some code.

[vikashg]

Hi @MMelQin,
So I looked up Enum which is similar to StrEnum. There are existing examples like

But I don't know how this will help or enhance the existing code. Please let me know and I will fix it.
Thanks
Vikash

[MMelQin]

Hi @MMelQin, So I looked up Enum which is similar to StrEnum. There are existing examples like

But I don't know how this will help or enhance the existing code. Please let me know and I will fix it. Thanks Vikash

Another issue is that the MD App SDK requires Python version >= 3.7, for which the StrEnum is NOT available. This is why MONAI Core had its impl of StrEnum, as Eric mentioned.

For our use case, the relevant magic strings had not been used in the App SDK or app code, I think we can safely use Enum or IntEnum for now. However, the monai.utils.enums.py has already ported the StrEnum and noticed this class is already imported and used in this change, we'll use StrEnum.

I have pushed a couple changes to define and use the InfererType StrEnum along with updating a couple more arguments, as well as updating the app code to demonstrate the use of the InfererType. Please also try out the update with your simple inference application.

[MMelQin]

LGTM

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication