From: E.L. W. <eg...@sc...> - 2002-09-03 06:56:51
|
On Monday 02 September 2002 23:07, Bradley Smith wrote: > My opinion of the patch is that it is difficient in a number of areas, and > is not ready to go into Jmol. Ok, those below are all valid points... But what is your point on inclusion of the extention in general? > 1. Crystal properties don't warrant a menu of their own. Rather than having > Crystal/Properties, it should be Extras/Crystal properties... > > 2. The menu item is activated regardless of whether crystal information is > read from the file. If caffeine.xyz is read, the crystal properties are > active with defaults that are certianly not going to be related to a > crystal structure of caffeine. In addition, the molecule explodes if the > Commit changes... button is clicked. > > 3. Like most of the dialogs in Jmol, it overuses the TitledBorder. Granted > the same design is used throughout Jmol, but better design should be used > for new dialogs. > > 4. Why is the total number of frames listed in the crystal properties? What > does that have to do with crystals properties? > > 5. Values entered into the dialog box are not checked for validity. > Incorrect values cause exceptions to be thrown. > > 6. The crystal properties dialog does not update if a molecule is read > while it is open. > > 7. The space group information is missing. > > These are just the first impressions obtained from using the new code. I > have not looked at the implementation itself. > > Personally, I don't understand the information presented in the crystal > properties. The information is different than that generally used with > protein crystallography, which uses cell dimensions (a, b, c) and angles > (alpha, beta, gamma), and the space group. Egon |