Skip to content

Commit

Permalink
Merge pull request #3176 from rmu75/rs/fix-3147
Browse files Browse the repository at this point in the history
fix tool parameter mismatch in StatMixin
  • Loading branch information
rene-dev authored Nov 25, 2024
2 parents b38ce55 + 2b910dc commit 8b50ad6
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion lib/python/rs274/interpret.py
Original file line number Diff line number Diff line change
Expand Up @@ -140,8 +140,9 @@ def __init__(self, s, r):
self.tools = list(s.tool_table)
self.random = r

def change_tool(self, idx):
def change_tool(self, tool_nr):
global tool_in_spindle
idx = self.get_index(tool_nr)
if self.random:
self.tools[0], self.tools[idx] = self.tools[idx], self.tools[0]
tool_in_spindle = idx
Expand All @@ -158,6 +159,14 @@ def get_tool(self, idx):
return tuple(self.tools[idx])
return empty_spindle_data

def get_index(self, tool_nr):
index = 1
for tool in self.tools[1:]:
if tool.id == tool_nr:
return index
index = index + 1
return 0

def get_external_angular_units(self):
return self.s.angular_units or 1.0

Expand Down

0 comments on commit 8b50ad6

Please sign in to comment.