brain-slam / brain-slam/slam

[MAINT] remove useless line in curvature_fit.py

Open
#82 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
22
Forks
37
PR merge metrics
No merged PRs in 30d

Description

hello,
In the curvature_fit.py function i notice one useless line :

line 35 if (i == 175):
print("Debug")
Moreover there are commented lines :
line23-24 . vertex_normals =
faces=
line 44 rotation_matrix =
line 53. neigh =
line 68 parameters =

It's a little confused to have commented lines that defines variable differently,
can we remove this lines ?

thanks for your help

Contributor guide

No contributing guide indexed for this repository

Research direction

Open curvature_fit.py and inspect the listed lines in the curvature-fitting function, especially the conditional Debug print and commented variable assignments. Remove the unused debug and commented lines, then verify that the function remains valid and contains no unintended references to those variables.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.