cmd-ntrf / jupyter-lmod

Jupyter plugin that provides a tab for TACC Lmod (https://github.com/TACC/Lmod)
MIT License
28 stars 13 forks source link

Refactor hidden module regex logic #48

Closed cmd-ntrf closed 2 years ago

cmd-ntrf commented 2 years ago

Instead of running the regex on all module names, we now verify if the module is hidden once we have verified that it matches the module regex. This simplifies considerably the logic.

This commit also replaces the call to module("list") by a more efficient split of the content of the LOADEDMODULES environment variable.

Fixes #47