e-Learning-by-SSE / nm-skill-lib

Apache License 2.0
0 stars 0 forks source link

Sanity Checks: Detect Paths to Cycled Skills #8

Closed Elscha closed 11 months ago

Elscha commented 11 months ago

nm-skill-lib offers already a function to detect cycles in Skills (nesting hierarchy) or in combination with LearningUnits. However, the SelfLearn Skill-editor will only show the root Skills by default and, therefore, may hide nested Skills involved in cycles.

To support the user, the editor should also provide a functionality to highlight all Skills, that contain a cycle below its hierarchy. This must also consider problematic Skills, that are nested below multiple Skills.

Given the example from below, there should be the following elements be highlighted:

Image

To address this, the nm-skill-lib should be extended by an alternative checking function, that takes only Skills as input and returns for all cycles also all nesting structures from toplevel Skills down to the cycles.

This issue replaces e-Learning-by-SSE/nm-concept#18

Elscha commented 11 months ago

Closed by 716948a