Hi @wyli and @Nic-Ma,
These transformations only have effect when they are run with nsys profile, and also the results need to be visualized by "Nsight Systems" in order to be checked, so I am not sure how to write a test case to work with MONAI CI/CD. I have tested them in my local machine, and they are working as expected. What do suggest to do?
Considering that it does not have any effect on the rest of MONAI components, is the local testing fine?

@wyli and @Nic-Ma, I added some unittests to make sure that these tranforms do not alter anything when they are used alone or in conjunction with other array or dictionary transforms.

@ericspod @rijobro @wyli ,

Do you guys have any other comments?

Thanks in advance.

@ericspod @rijobro @wyli ,

Do you guys have any other comments?

Thanks in advance.

I'd appreciate it if you can give me any comment that you have.
Thanks.

Thanks for the PR, my feeling is that the NVTX tools are not well-known in our community. Shall we have some tutorials first, then revisit this PR and move the reusable code blocks from the tutorials into the core codebase? I don't know why these are wrapped as transforms instead of network layers, but I guess tutorials would clarify this easily.

Thanks for the PR, my feeling is that the NVTX tools are not well-known in our community. Shall we have some tutorials first, then revisit this PR and move the reusable code blocks from the tutorials into the core codebase? I don't know why these are wrapped as transforms instead of network layers, but I guess tutorials would clarify this easily.

@wyli, tutorial would be great to teach the community on how to use Nsight System and NVTX.
Nsight is able to profile the network by default, so there is no need to wrap them as network. To profile a chain of transforms, you should either put NVTX in MONAI source code for each transform, or wrap them as transforms to be able to put them where is need in a transform pipeline. This PR is doing the latter, which is not intrusive and does not affect any other components in MONAI.

Hi @drbeh ,

Thanks for bringing this interesting feature to MONAI.
To verify and test the APIs, I would suggest you to add a simple tutorial notebook in the MONAI tutorials repo, and put the link in the doc-string of this module. Then users can easily get start to use it and we can also easily verify & test it.

Thanks.

Hi @drbeh ,

As I said in previous comment, I agree with @wyli , we should make a tutorial first and put the link in the doc-string of this module.

Thanks.

Hi @drbeh ,

As I said in previous comment, I agree with @wyli , we should make a tutorial first and put the link in the doc-string of this module.

Thanks.

@wyli, @Nic-Ma, I definitely agree with having a tutorial on this but the tutorial will depend on these components, so shouldn't we first merge this and then use this in a tutorial?!

Similar to the other components where we first have implemented in MONAI (like other transforms and models), and then created tutorial for spleen segmentation for example, right?

Hi @drbeh ,

I would suggest you to develop a tutorial notebook PR in the tutorials repo, then let's merge these 2 PRs together.

Thanks.

Hi @drbeh ,

I would suggest you to develop a tutorial notebook in the tutorials repo, then let's merge these 2 PRs together.

Thanks.

Sure, I can do that.

Hi @Nic-Ma , @wyli

I have created a notebook and necessary codes for profiling digital pathology transformation pipeline using these NVTX transforms, and NVIDIA Nsight System. Please check it out: Project-MONAI/tutorials#305

Thanks

/build