huppertt / nirs-toolbox

Toolbox for fNIRS analysis
89 stars 61 forks source link

Module: RemovetooLongDistance() constructor issue #50

Open ouhscgl opened 2 months ago

ouhscgl commented 2 months ago

Issue Brief Description Due to assumed nomenclature change in line 7 the method function is named RemovetooLongSeparation but the class is named RemovetooLongDistance. This discrepancy disallows chaining this module into a pipeline, but works when assigned first to the job system.

Reproducing the Issue

  1. Attempt using job = nirs.modules.RemovetooLongDistance(job). Always reproducible.

Suggested Fix from:

7         function obj = RemovetooLongSeparation ( prevJob )

to:

7        function obj = RemovetooLongDistance( prevJob )

Fix Brief Description Corrects the contructor.