Skip to content

Only Hide Training area when zero tasks configured#769

Merged
SachidanandAlle merged 1 commit intomainfrom
slicer_fix
May 5, 2022
Merged

Only Hide Training area when zero tasks configured#769
SachidanandAlle merged 1 commit intomainfrom
slicer_fix

Conversation

@SachidanandAlle
Copy link
Copy Markdown
Collaborator

@SachidanandAlle SachidanandAlle commented May 5, 2022

Signed-off-by: Sachidanand Alle [email protected]

Put Training items under widget and hide that instead of hiding entire active learning section

This should solve: #693 (comment)

@jlvahldiek ^^

@SachidanandAlle SachidanandAlle requested a review from diazandr3s May 5, 2022 13:35
@SachidanandAlle SachidanandAlle added the 0.4.0 Targeted to Release version 0.4 label May 5, 2022
@SachidanandAlle SachidanandAlle merged commit 1a4defa into main May 5, 2022
@SachidanandAlle SachidanandAlle deleted the slicer_fix branch May 5, 2022 13:48
@jlvahldiek
Copy link
Copy Markdown
Contributor

Thanks again for this quick fix.

Could you give me information when or at what timepoints updated versions are available via Slicer's Extensions Manager? Is there some kind of automatic upload or do you submit major releases?

@SachidanandAlle
Copy link
Copy Markdown
Collaborator Author

i think it's nightly build.. but i guess every nightly build doesn't push/update to extension manager.. u have to download and install slicer again.. may be try tomorrow.. @lassoan can confirm.. i remember there were some CI/CD links to see

@lassoan
Copy link
Copy Markdown
Collaborator

lassoan commented May 5, 2022

Extensions are updated nightly for the latest Slicer Stable Release. See details here: https://slicer.readthedocs.io/en/latest/user_guide/extensions_manager.html#update-extensions

However, nightly update of the Slicer Stable Release has stopped a few days ago, in preparation for the new Slicer5 release.

If you want to use the very latest extension version now then you need to install the very latest Slicer Preview Release. For each Slicer Preview Release the extensions are built once, on the night the release is created.

markus-hinsche pushed a commit to rAIdiance/MONAILabel that referenced this pull request May 12, 2022
SachidanandAlle added a commit that referenced this pull request May 19, 2022
* Only Hide Training area when zero tasks configured (#769)

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Nuclick train (#772)

* Draft Training workflow for NuClick

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* Fix nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* rename transform

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* use monai bunet for nuclick

Signed-off-by: Sachidanand Alle <[email protected]>

* fix log

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add unit test for NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>

* Polish

Signed-off-by: Markus Hinsche <[email protected]>

* Add labels to data object, Check for label list

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* update docstrings

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* change writer selection

write_seg_nrrd() is only needed for 4D multi-channel label arrays. Everything else can be handled by ITK of nifty writer.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* polish

write_seg_nrrd() will only be used for 4D multi-channel label arrays. Thus, removed unnecessary code.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix typo

Signed-off-by: Markus Hinsche <[email protected]>

* Add pynrrd==0.4.2 to setup.cfg

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* fix check for multi-channel label

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* fix unit test

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix multichannel problem

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* Reformat code

Signed-off-by: Markus Hinsche <[email protected]>

* Make mypy happy

Signed-off-by: Markus Hinsche <[email protected]>

* Fix merge conflict

Signed-off-by: Markus Hinsche <[email protected]>

* Use logger.warning instead logger.debug

Signed-off-by: Markus Hinsche <[email protected]>

* Add Active Learning strategies to DeepEdit (#782)

* Add Active Learning strategies to DeepEdit

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update readme - commands Active Learning strategies

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Prepare MONAI Label for new monai - DeepEdit transforms (#781)

* Prepare MONAI Label for new monai - DeepEdit transforms

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Add deprecated messages - DeepEdit transforms - interaction

Signed-off-by: Andres Diaz-Pinto <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Mention python versions which are supported (#786)

Signed-off-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add original labels option Slicer UI (#785)

* Add original labels option Slicer UI

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update Slicer module

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix: Check first dimension for multichannel

Signed-off-by: Markus Hinsche <[email protected]>

* Polish unit test to be more specific about dimensions

Signed-off-by: Markus Hinsche <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Co-authored-by: Andres Diaz-Pinto <[email protected]>
DaoMaoDuc pushed a commit to jlvahldiek/MONAILabel that referenced this pull request Jun 1, 2022
DaoMaoDuc pushed a commit to jlvahldiek/MONAILabel that referenced this pull request Jun 1, 2022
* Only Hide Training area when zero tasks configured (Project-MONAI#769)

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Nuclick train (Project-MONAI#772)

* Draft Training workflow for NuClick

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* Fix nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* rename transform

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* use monai bunet for nuclick

Signed-off-by: Sachidanand Alle <[email protected]>

* fix log

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add unit test for NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>

* Polish

Signed-off-by: Markus Hinsche <[email protected]>

* Add labels to data object, Check for label list

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* update docstrings

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* change writer selection

write_seg_nrrd() is only needed for 4D multi-channel label arrays. Everything else can be handled by ITK of nifty writer.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* polish

write_seg_nrrd() will only be used for 4D multi-channel label arrays. Thus, removed unnecessary code.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix typo

Signed-off-by: Markus Hinsche <[email protected]>

* Add pynrrd==0.4.2 to setup.cfg

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* fix check for multi-channel label

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* fix unit test

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix multichannel problem

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* Reformat code

Signed-off-by: Markus Hinsche <[email protected]>

* Make mypy happy

Signed-off-by: Markus Hinsche <[email protected]>

* Fix merge conflict

Signed-off-by: Markus Hinsche <[email protected]>

* Use logger.warning instead logger.debug

Signed-off-by: Markus Hinsche <[email protected]>

* Add Active Learning strategies to DeepEdit (Project-MONAI#782)

* Add Active Learning strategies to DeepEdit

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update readme - commands Active Learning strategies

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Prepare MONAI Label for new monai - DeepEdit transforms (Project-MONAI#781)

* Prepare MONAI Label for new monai - DeepEdit transforms

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Add deprecated messages - DeepEdit transforms - interaction

Signed-off-by: Andres Diaz-Pinto <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Mention python versions which are supported (Project-MONAI#786)

Signed-off-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add original labels option Slicer UI (Project-MONAI#785)

* Add original labels option Slicer UI

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update Slicer module

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix: Check first dimension for multichannel

Signed-off-by: Markus Hinsche <[email protected]>

* Polish unit test to be more specific about dimensions

Signed-off-by: Markus Hinsche <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Co-authored-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Duc <[email protected]>
Douwe-Spaanderman pushed a commit to Douwe-Spaanderman/MONAILabel that referenced this pull request Dec 9, 2022
Douwe-Spaanderman pushed a commit to Douwe-Spaanderman/MONAILabel that referenced this pull request Dec 9, 2022
* Only Hide Training area when zero tasks configured (Project-MONAI#769)

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Nuclick train (Project-MONAI#772)

* Draft Training workflow for NuClick

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* Fix nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* rename transform

Signed-off-by: Sachidanand Alle <[email protected]>

* Sync up changes for nuclick training

Signed-off-by: Sachidanand Alle <[email protected]>

* use monai bunet for nuclick

Signed-off-by: Sachidanand Alle <[email protected]>

* fix log

Signed-off-by: Sachidanand Alle <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add unit test for NRRD writer

Signed-off-by: Markus Hinsche <[email protected]>

* Polish

Signed-off-by: Markus Hinsche <[email protected]>

* Add labels to data object, Check for label list

Signed-off-by: Markus Hinsche <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* update docstrings

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* change writer selection

write_seg_nrrd() is only needed for 4D multi-channel label arrays. Everything else can be handled by ITK of nifty writer.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* polish

write_seg_nrrd() will only be used for 4D multi-channel label arrays. Thus, removed unnecessary code.

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix typo

Signed-off-by: Markus Hinsche <[email protected]>

* Add pynrrd==0.4.2 to setup.cfg

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* fix check for multi-channel label

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* fix unit test

Signed-off-by: Janis Vahldiek <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix multichannel problem

Signed-off-by: Markus Hinsche
Signed-off-by: Markus Hinsche <[email protected]>

* Reformat code

Signed-off-by: Markus Hinsche <[email protected]>

* Make mypy happy

Signed-off-by: Markus Hinsche <[email protected]>

* Fix merge conflict

Signed-off-by: Markus Hinsche <[email protected]>

* Use logger.warning instead logger.debug

Signed-off-by: Markus Hinsche <[email protected]>

* Add Active Learning strategies to DeepEdit (Project-MONAI#782)

* Add Active Learning strategies to DeepEdit

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update readme - commands Active Learning strategies

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Prepare MONAI Label for new monai - DeepEdit transforms (Project-MONAI#781)

* Prepare MONAI Label for new monai - DeepEdit transforms

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Add deprecated messages - DeepEdit transforms - interaction

Signed-off-by: Andres Diaz-Pinto <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Mention python versions which are supported (Project-MONAI#786)

Signed-off-by: SACHIDANAND ALLE <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Add original labels option Slicer UI (Project-MONAI#785)

* Add original labels option Slicer UI

Signed-off-by: Andres Diaz-Pinto <[email protected]>

* Update Slicer module

Signed-off-by: Andres Diaz-Pinto <[email protected]>
Signed-off-by: Markus Hinsche <[email protected]>

* Fix: Check first dimension for multichannel

Signed-off-by: Markus Hinsche <[email protected]>

* Polish unit test to be more specific about dimensions

Signed-off-by: Markus Hinsche <[email protected]>

Co-authored-by: SACHIDANAND ALLE <[email protected]>
Co-authored-by: Janis Vahldiek <[email protected]>
Co-authored-by: Andres Diaz-Pinto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.4.0 Targeted to Release version 0.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants