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.

Change History (1)

comment:1 by Tom Goddard, 6 years ago

Resolution: fixed
Status: assignedclosed

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.

Note: See TracTickets for help on using tickets.