#878 closed defect (not a bug)
Command keyword error
| Reported by: | Tom Goddard | Owned by: | Greg Couch |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | Command Line | Version: | |
| Keywords: | Cc: | meng@… | |
| Blocked By: | Blocking: | ||
| Notify when closed: | Platform: | all | |
| Project: | ChimeraX |
Description (last modified by )
Command keyword argument abbreviated to the first word, for example, ballScale in the following style command, is giving an error:
style #1 ball 1 Expected a keyword
Change History (6)
comment:1 by , 8 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 8 years ago
| Description: | modified (diff) |
|---|---|
| Resolution: | → not a bug |
| Status: | assigned → closed |
comment:3 by , 8 years ago
| Description: | modified (diff) |
|---|
comment:4 by , 8 years ago
| Cc: | added |
|---|
Elaine, any ideas how to change the keyword names of the style command so that
style #1 stick 0.5
meaning to change stickRadius does not get tripped up interpreting it as "style #1 stick" with some extra 0.5 argument giving and error? We could leave it as is and require "style #1 stickR 0.5". Likewise "ballScale" option can't be shortened to ball because it gets confused with "ball" style.
follow-up: 5 comment:5 by , 8 years ago
I’ve been using “stickrad” and “ballsc” which I didn’t think were too bad. Can’t think of sufficiently descriptive keywords that avoid the problem, although I might change “pseudobondRadius” to “pbondRadius”. The problem is that the same command is used to change between styles and to change the parameters of an individual style. The alternative is to have a separate “size” or “radius” command (didn’t we have that originally?), i.e. radius atomspec N [atoms | sticks | pbonds]
follow-up: 6 comment:6 by , 8 years ago
Never liked the size settings in the style command since I don’t think of “style” when I want to change a size so it is a struggle to find the command. So I’ve added a size command and moved the style size options to the size command.
Note:
See TracTickets
for help on using tickets.
Try
style #1 ba 1and you'll see that the ba is expanded to ball, i.e., an atom style. To avoid the conflict, either the ballScale keyword or the ball style need to be renamed.