Skip to content

Commit

Permalink
fix: Added sigma restraining condition
Browse files Browse the repository at this point in the history
  • Loading branch information
JoaoDell committed Jul 27, 2023
1 parent 7ecefeb commit 68e520e
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions fury/actors/effect_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,8 @@ def kde(self,
sigmas = np.array(sigmas)

Check warning on line 239 in fury/actors/effect_manager.py

View check run for this annotation

Codecov / codecov/patch

fury/actors/effect_manager.py#L239

Added line #L239 was not covered by tests
if sigmas.shape[0] != 1 and sigmas.shape[0] != points.shape[0]:
raise IndexError("sigmas size must be one or points size.")

Check warning on line 241 in fury/actors/effect_manager.py

View check run for this annotation

Codecov / codecov/patch

fury/actors/effect_manager.py#L241

Added line #L241 was not covered by tests
if np.min(sigmas) <= 0:
raise ValueError("sigmas can't have zero or negative values.")

Check warning on line 243 in fury/actors/effect_manager.py

View check run for this annotation

Codecov / codecov/patch

fury/actors/effect_manager.py#L243

Added line #L243 was not covered by tests

varying_dec = """

Check warning on line 245 in fury/actors/effect_manager.py

View check run for this annotation

Codecov / codecov/patch

fury/actors/effect_manager.py#L245

Added line #L245 was not covered by tests
varying float out_sigma;
Expand Down

0 comments on commit 68e520e

Please sign in to comment.