Opened 6 years ago
Closed 6 years ago
#1936 closed defect (fixed)
"view sel" with volume isosurf selected causes traceback
Reported by: | Elaine Meng | Owned by: | Tom Goddard |
---|---|---|---|
Priority: | moderate | Milestone: | |
Component: | Volume Data | Version: | |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Notify when closed: | Platform: | all | |
Project: | ChimeraX |
Description
e.g.
open 1pho
molmap helix 5
select #2
view sel
Traceback (most recent call last):
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/cmd_line/tool.py", line 254, in execute
cmd.run(cmd_text)
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/core/commands/cli.py", line 2631, in run
result = ci.function(session, kw_args)
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/std_commands/view.py", line 55, in view
view_objects(objects, v, clip, cofr, pad)
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/std_commands/view.py", line 63, in view_objects
b = disp.bounds()
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/core/objects.py", line 211, in bounds
ib = copies_bounding_box(b, m.positions.masked(minst))
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/core/geometry/place.py", line 675, in masked
p = Places(place_array=self.array()[mask])
IndexError: arrays used as indices must be of integer (or boolean) type
IndexError: arrays used as indices must be of integer (or boolean) type
File "/Users/meng/Desktop/ChimeraX.app/Contents/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/chimerax/core/geometry/place.py", line 675, in masked
p = Places(place_array=self.array()[mask])
See log for complete Python traceback.
Fixed.
Code that handled selected objects (Objects class) did not properly handle Drawing.display_positions being None. Changed so that Drawing.display_positions is always a bool array.