[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
--
Reply to: [list | sender only]
Re: [ddlm-group] Split _model_site.adp_eigen_system intoeigen_vector and eigen_value
- To: ddlm-group <ddlm-group@iucr.org>
- Subject: Re: [ddlm-group] Split _model_site.adp_eigen_system intoeigen_vector and eigen_value
- From: James H <jamesrhester@gmail.com>
- Date: Thu, 1 Jul 2021 13:02:39 +1000
- In-Reply-To: <CH2PR04MB6950B98360FA79EDF1413236E0579@CH2PR04MB6950.namprd04.prod.outlook.com>
- References: <CAM+dB2esakiu7pvDVyQGPpec9sgqvdCPa8uaBKkQft6xCgU2Gw@mail.gmail.com><CH2PR04MB6950B98360FA79EDF1413236E0579@CH2PR04MB6950.namprd04.prod.outlook.com>
Dear Group,
There having been no objections, I will proceed as planned.
best wishes,
James.
On Sat, 8 May 2021 at 00:27, Bollinger, John C <John.Bollinger@stjude.org> wrote:
Dear group,
I am in favor of addressing this issue by splitting _model_site.adp_eigen_system into eigen_vector and eigen_value, as proposed.
John
--John C. Bollinger, Ph.D., RHCSAComputing and X-ray ScientistDepartment of Structural BiologySt. Jude Children's Research Hospital
From: ddlm-group <ddlm-group-bounces@iucr.org> on behalf of James H <jamesrhester@gmail.com>
Sent: Friday, May 7, 2021 12:10 AM
To: ddlm-group <ddlm-group@iucr.org>
Subject: [ddlm-group] Split _model_site.adp_eigen_system into eigen_vector and eigen_valueÂCaution: External Sender. Do not open unless you know the content is safe.
Dear DDLm gurus,
As part of tidying up the core CIF dictionary in preparation for Vol G publication, we have come across the following fundamental issue: only one unit can be specified in a definition, which means that those units apply to all elements of a matrix or list. However, _model_site.adp_eigen_system bundles both eigenvalues (dimensionless) and eigenvectors (dimensioned) in one matrix. This is purely for computational convenience, because eigen-vectors and -values are efficiently computed together. Note that this data name cannot and did not exist in the old DDL1 dictionary, so we have considerable freedom to fiddle.
I would like to suggest that we split this into two data names corresponding to the eigenvectors and eigenvalues. The dREL definitions will precisely define how they are supposed to be used and how they relate to one another.
My main reason for approaching it this way is that compound data names whose components are handled differently by code should be split into separate items to reflect the implied distinct meanings. We have done this for all other cases in the core dictionary. Differing units is an obvious criterion for separating components, as different units implies that these components have different meanings.
Other alternatives that I find less attractive are:(1) Enhance the DDLm _units.code attribute to somehow handle compound data items - complex and disproportionate for a single data name (there are no others in cif core)(2) Add a _units.code enumerated value of something like 'derived' which signals that the units should be derived from the dREL calculation, which in this case would simply join the eigenvectors and eigenvalues together. Possible but excludes software that doesn't want to interpret dREL just to get units.
Please let me know your thoughts. If none are forthcoming, I'll go with my preference.
James.--
T +61 (02) 9717 9907
F +61 (02) 9717 3145
M +61 (04) 0249 4148
Email Disclaimer: www.stjude.org/emaildisclaimer
Consultation Disclaimer: www.stjude.org/consultationdisclaimer
T +61 (02) 9717 9907
F +61 (02) 9717 3145
M +61 (04) 0249 4148
F +61 (02) 9717 3145
M +61 (04) 0249 4148
_______________________________________________ ddlm-group mailing list ddlm-group@iucr.org http://mailman.iucr.org/cgi-bin/mailman/listinfo/ddlm-group
Reply to: [list | sender only]
- References:
- Prev by Date: Re: [ddlm-group] Revising dictionary_valid category
- Next by Date: Re: [ddlm-group] Revising dictionary_valid category
- Prev by thread: Re: [ddlm-group] Split _model_site.adp_eigen_system intoeigen_vector and eigen_value
- Next by thread: [ddlm-group] Trailing whitespace in CIF2.0 text fields: statementfrom draft syntax chapter of Vol G
- Index(es):