Update profiling code with NVTX handlers#310
Conversation
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
|
As I mentioned in #308, this notebook now tries to download two large histopathology images. Although the runner should work but it might take a while depending on the server speed (about 1 hour). Do you want me to exclude it from |
|
Hi @wyli , I feel 1 hour is acceptable for cron job, but it should be stable enough. Thanks. |
|
I saw that the cron job has 150min timeout for all the notebook. I'll move things to our google drive (instead of outside ftp) to speed up the download, and will change this notebook accordingly. |
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
|
@Nic-Ma @wyli, I don't have write access to this repo so can any of you merge it. Thanks |
wyli
left a comment
There was a problem hiding this comment.
Thanks for enhancing the tests!
* Update profiling code with NVTX handlers Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com> * Change a range name Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com> * Enable test for camelyon and add it to doesnt_contain_max_epochs Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com> * Add data prepration and dependencies Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com> * minor changes Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com> * Update with a smaller gdrive-hosted WSI Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>

Description
This PR update the NVTX profiling to include NVTX handlers, and also update the
RangePush/RangePopto the simpler interface ofRange.This requires MONAI/PR2765 to be merged first.
Status
Ready
Checks