# 2024.04.23 Weekly Meeting
**URL:** https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659
**Category:** Weekly meetings
**Created:** [April 22, 2024, 8:31pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659 "2024-04-22T20:31:37Z")
**Posts on this page:** 6
**Page:** 1
### Author:  [@Sam\_Horvath](https://discourse.slicer.org/u/Sam_Horvath)
#### Post date: [April 22, 2024, 8:31pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/1 "2024-04-22T20:31:37Z")
Tomorrow, we will be having our next weekly hangout at **10:00 AM ET until 11:00 AM ET**.
Anyone is welcome to join at this link: [https://bit.ly/slicer-googlemeet-hosted-by-kitware](https://bit.ly/slicer-googlemeet-hosted-by-kitware)
* * *
**Agenda:**
Please post to this thread to put a topic on the agenda! We will try to prioritize agenda items during the meeting.
* * *
Thanks
Sam and J-Christophe
---
### Author:  [@evaherbst](https://discourse.slicer.org/u/evaherbst)
#### Post date: [April 23, 2024, 11:10am UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/2 "2024-04-23T11:10:24Z")
I would like to discuss 1) modifying the TotalSegmentator widget as suggested in [this thread](https://discourse.slicer.org/t/running-totalsegmentator-from-python-console/35461/13) and 2) [fixing my issue](https://discourse.slicer.org/t/totalsegmentator-extra-task-license-not-saved-in-slicer-6-2/35622/3) with TotalSegmentator license not being saved
---
### Author:  [@jcfr](https://discourse.slicer.org/u/jcfr)
#### Post date: [April 23, 2024, 1:20pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/3 "2024-04-23T13:20:36Z")
Discuss and finalize these two pull requests:
- [ExtensionsIndex#2011](https://github.com/Slicer/ExtensionsIndex/pull/2011): CI: Transition from s4ext to json
- [Slicer#7629](https://github.com/Slicer/Slicer/pull/7629): Switch extension index entry format from “s4ext” to “json”
Notes:
- Added support for using a restricted set of HTML tags in extension description through [girder/slicer\_package\_manager#123](https://github.com/girder/slicer_package_manager/pull/123) and [KitwareMedical/slicer-extensions-webapp#159](https://github.com/KitwareMedical/slicer-extensions-webapp/pull/159).
Related posts:
- [Introduction of "tiers" for Slicer extensions](https://discourse.slicer.org/t/introduction-of-tiers-for-slicer-extensions/34870)
---
### Author:  [@allemangd](https://discourse.slicer.org/u/allemangd)
#### Post date: [April 23, 2024, 1:31pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/4 "2024-04-23T13:31:28Z")
Discuss revision of proposal for scripted modules requirements.txt support following last week’s feedback.
>
>
> Continuation of #7697; tightly coupled to #7171 and #6913.
>
> After discussion a…nd feedback at the weekly hangout Apr 16 (see https://github.com/Slicer/Slicer/issues/7697#issuecomment-2059271196 and https://github.com/Slicer/Slicer/issues/7697#issuecomment-2059273077) I think my first issue conflated three separate tasks:
>
> \- Declare and install dependencies
> \- Run modules in an isolated subprocess (CLI modules)
> \- Run modules in an isolated environment (venv, conda, poetry, etc)
>
> I'm interested in tackling only the first of these, and since CLI have questionable value anyway it doesn't make sense to require CLI usage to get the benefit here. Really, what we want is a way for scripted module's logic to depend on pypi packages, and only install those packages when they are needed. Doing this purely in the Python layer allows all scripted modules to benefit.
>
> \## Draft implementation and features
>
> I have a Python 3.9 draft implementation of a \`LazyImportGroup\` context manager, based on the \[\`LazyLoader\`\](https://docs.python.org/3/library/importlib.html#importlib.util.LazyLoader) and \[\`lazy\_import\` recipe\](https://docs.python.org/3/library/importlib.html#implementing-lazy-imports) in the standard library \`importlib\` documentation.
>
> https://github.com/allemangD/lazyloader - see the doctests for \`LazyImportGroup\` for more details.
>
> The important principles here are:
>
> \- Consolidate. Keep all \`import\` statements at top-level, all dependencies in \`requirements.txt\`.
> \- Lazy. Do not install or import expensive libraries until they are used.
> \- Reliable. Do not import libraries unless they are guaranteed to be available in a user's environment.
>
> These were not mentioned during the discussion last week, but I think they are important:
>
> \- Support static analysis.
> \- Be explicit about when "magic" occurs.
>
> \---
>
> Usage requires the extension developer place a \`requirements.txt\` in a python package, installed via \`RESOURCES\` argument of the existing CMake macros. The \[anchor and path\](https://docs.python.org/3/library/importlib.resources.html#importlib.resources.files) for that \`requirements.txt\` resource are given as an argument to \`LazyImportGroup\` context manager, wrapping top-level imports.
>
> \`\`\`py
> with LazyImportGroup('lib:requirements.txt'):
> import lib.logic
> import itk
> \`\`\`
>
> These imports are real import statements that are detected by static analysis, and run-time magic only occurs with imports within the context manager. These imports produce proxy modules that defer install and real import until first usage:
>
> \`\`\`py
> def onApplyButton(self):
> lib.logic.do\_the\_task(...)
> # ^ attribute access triggers install and import
>
> def onComputeButton(self):
> itk.Fpfh....
> # ^ attribute access triggers install and import
> \`\`\`
>
> \---
>
> Modules imported within a \`LazyImportGroup\` can safely have bare top-level imports of their dependencies. This is probably recommended usage for this tool:
>
> \`\`\`
> qt-scripted-modules
> |- libCompute
> | |- \_\_init\_\_.py
> | |- requirements.txt
> | \\- logic.py
> \\- Compute.py
> \`\`\`
>
> Above, \`\_\_init\_\_.py\` and \`train\` can safely include their dependencies at top level, as long as \`Compute.py\` imports them in a lazy group:
>
> \`\`\`
> \# requirements.txt
> A
> B
> \`\`\`
>
> \`\`\`py
> \# train.py
> import A
> \`\`\`
>
> \`\`\`py
> \# \_\_init\_\_.py
> import B
> from .logic import ...
> \`\`\`
>
> \`\`\`py
> \# Compute.py
> with LazyImportGroup('libCompute:requirements.txt'):
> import libCompute
> import libCompute.logic
> \`\`\`
>
> \---
>
> Only the first attribute access of a module in a group will trigger the installation for all imports in that group. There may be multiple groups with different requirements to denote multiple feature sets, and only the used features will be installed. Each group is installed at most a single time.
>
> \`\`\`py
> with LazyImportGroup('lib:training-requirements.txt'):
> import X
> import Y
>
> with LazyImportGroup('libinference:requirements.txt'):
> import Z
> \`\`\`
>
> The first attribute access of X or Y will install \`training-requirements.txt\`, and the first attribute access of Z will install \`inference-requirements.txt\`.
>
> A fast \`pip install --no-deps --dry-run --report\` check is first used to ensure all the items in \`requirements.txt\` are satisfied. If so, no action is taken, the module is imported, and the proxy module is updated. If not, the requirements are first installed with \`pip install -r\`.
>
> The dry-run report is used to produce a summary of packages to be installed. The Slicer implementation should show this to the user with \`slicer.util.confirmOkCancelDisplay\`.
>
> Not implemented in the draft, but the report and install steps could accept a \`-c constraints.txt\` argument to pin Slicer libraries like \`numpy\`, \`SimpleITK\`, etc. Also the requirements of other groups could be inspected to detect incompatibilities with messages like: "Module X wants to install package Y which is incompatible with module Z. Proceed?"
>
> \---
>
> I've tested these proxy objects on plain python packages, loadable packages, and even \`itk\`'s complex lazy-loaded metapackage. It works on all of these, although there are some caveats on object identity and safe import order. See the doctests for details.
>
> \## Outstanding questions
>
> \- Where to place this? Suggest \`slicer.LazyImportGroup\`.
> \- Bikeshed the name? Signature? It also does pip installations.
> \- Best practices on confirmation dialog?
> \- Should constraints files or compatibility checks between modules be part of this initial feature? Or should they be delayed till we see usage in practice.
>
> \---
>
> From prior discussions: cc @jcfr @pieper @lassoan @ebrahimebrahim
---
### Author:  [@jcfr](https://discourse.slicer.org/u/jcfr)
#### Post date: [April 23, 2024, 5:28pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/5 "2024-04-23T17:28:04Z")
### Meeting notes
- Comment on SlicerWMA extension renaming
- See [https://github.com/Slicer/ExtensionsIndex/pull/2039#discussion\_r1576314015](https://github.com/Slicer/ExtensionsIndex/pull/2039#discussion_r1576314015)
- TotalSegmentator
- Discussed [this thread](https://discourse.slicer.org/t/running-totalsegmentator-from-python-console/35461/13)
- Suggested path forward: Add an interactive option
- artisynth
- Combined MultiBody/FEM Simulation
- [ArtiSynth | Software / Documentation](https://www.artisynth.org/Software/Documentation)
- [artisynth · GitHub](https://github.com/artisynth)
- [slicer-media-assets](https://github.com/Slicer/slicer-media-assets)
- Added `QStyled` variation
- Transition to `README.md`
- Use [https://hackmd.io/](https://hackmd.io/) by first copying `README.txt` and modiyfing it
- Extension Index: Transition from s4ext to json. Pull requests ready for review
- [ExtensionsIndex#2011](https://github.com/Slicer/ExtensionsIndex/pull/2011): CI: Transition from s4ext to json
- [Slicer#7629](https://github.com/Slicer/Slicer/pull/7629): Switch extension index entry format from “s4ext” to “json”
- Proposal for scripted modules `requirements.txt` support
- [Slicer#7707](https://github.com/Slicer/Slicer/issues/7707): Allow scripted modules to declare and lazily install pip requirements
- Follow-up notes posted at [Allow scripted modules to declare and lazily install pip requirements · Issue #7707 · Slicer/Slicer · GitHub](https://github.com/Slicer/Slicer/issues/7707#issuecomment-2072745362)
- SlicerTelemetry
- Working on identifying URL for mapping with specific “action” and associated data to log
- Training
- Example of portal: [http://www.graysvertebrateanatomy.com/work/3d-imaging-resources/](http://www.graysvertebrateanatomy.com/work/3d-imaging-resources/)
---
### Author:  [@pieper](https://discourse.slicer.org/u/pieper)
#### Post date: [April 23, 2024, 5:53pm UTC](https://discourse.slicer.org/t/2024-04-23-weekly-meeting/35659/6 "2024-04-23T17:53:59Z")
> [@jcfr](#):
>
> - Training
> - Example of portal: [http://www.graysvertebrateanatomy.com/work/3d-imaging-resources/](http://www.graysvertebrateanatomy.com/work/3d-imaging-resources/)
This portal is really nice. Be sure to check out this Quick Guide document. It nicely summarizes the steps in a workflow and each slicer icon is a deep link into a demo video illustrating the use of the tool. It’s designed for biology samples but it’s a great resource for anyone doing segmentation and analysis:
[http://www.graysvertebrateanatomy.com/\_\_static/c0e61a322fc1f771762f9a4d60fbf7ee/gray-3d-slicer-quick-guide.pdf?dl=1](http://www.graysvertebrateanatomy.com/__static/c0e61a322fc1f771762f9a4d60fbf7ee/gray-3d-slicer-quick-guide.pdf?dl=1)
