TypeError setting framerate

[Zach Pearson]

Just another one of these Python 3.11-no-longer-does-type-coercion bugs

File "/Users/zjp/git/rbvi/ChimeraX/src/bundles/core/src/updateloop.py", line 115, in set_redraw_interval

t.start(self.redraw_interval)

TypeError: arguments did not match any overloaded call:

start(self, msec: int): argument 1 has unexpected type 'float'
start(self): too many arguments

Could just truncate redraw_interval with int(redraw_interval) here. Tom if you agree I'll make the change

[Eric Pettersen]

Could also use round() instead of int().

--Eric

[Tom Goddard]

What did the implicit casting from float to int in Python 3.9 do? I'd suggest doing the same as what the implicit cast did to maintain compatible behavior with the past.

[Zach Pearson]

I think ​https://bugs.python.org/issue37999 implies int() is the equivalent instead of round() because the underlying method was int() and the bug title is 'No longer use implicit convertion to int with loss'. I think loss means truncate here.

[Tom Goddard]

Testing in ChimeraX 1.6 shows timer.start(0.9) results in timer.interval() = 0. So cast to int is the right behavior.

[Zach Pearson]

​Will be fixed tomorrow