[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
My first reaction is that "dataname.a" is problematic. The presence of the period demands that ["dataname.a"] syntax be used. And it suggest poor parsing of the data labels. But it might be the simplest option.
Reply to: [list | sender only]
Re: Draft JSON specification for CIF
- Subject: Re: Draft JSON specification for CIF
- From: Robert Hanson <hansonr@xxxxxxxxxx>
- Date: Wed, 12 Apr 2017 06:36:51 -0500
- DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=stolaf.edu; s=stolaf;h=mime-version:in-reply-to:references:from:date:message-id:subject:to;bh=CGQutbMlRev0U1Tv470hW0rsK7PHyzIN8+Tx6h0gIr4=;b=NLXBMCp1/WVihXowdArMgnSSy9cK1uw/rwU/j2S+P1mdmJrLqVUN0FgSKv2Fm+sZAVr17yjVekuZ9826g4/PBJ0Kqg7s86qXbrqpzMse87gP2WTXYa0h5kWf3rgRNGuX+FDb86cOIz0xAIUNl6GRwrO8M3E7IzGPHqZCFL7Qps0=
- In-Reply-To: <CAM+dB2fszww=4A_w6evqg=5O9KKLnujajmg_SPSX=hCRQiBPtg@mail.gmail.com>
- References: <CAM+dB2fszww=4A_w6evqg=5O9KKLnujajmg_SPSX=hCRQiBPtg@mail.gmail.com>
I'd like to see an actual CIF or mmCIF file turned into JSON by this spec before judging.
But I'm pretty sure this is exactly what Jmol does already.
My first reaction is that "dataname.a" is problematic. The presence of the period demands that ["dataname.a"] syntax be used. And it suggest poor parsing of the data labels. But it might be the simplest option.
An important aspect it seems to me is how to parse data names that include "." vs. "_". Jmol ignores this difference completely, partially because mmCIF and CIF seem to differ some on this. (I can't remember what the IUCr spec says on "." vs. "_".)
The CIF file as shown is invalid. I presume there is an implied CIF2 header?
I would recommend REQUIRING a CIF2 header.
Bob Hanson
_______________________________________________ cif-developers mailing list cif-developers@iucr.org http://mailman.iucr.org/cgi-bin/mailman/listinfo/cif-developers
Reply to: [list | sender only]
- Follow-Ups:
- Re: Draft JSON specification for CIF (Andrius Merkys)
- Re: Draft JSON specification for CIF (James Hester)
- References:
- Draft JSON specification for CIF (James Hester)
- Prev by Date: Standard uncertainties (SU) in the DDLm dictionary
- Next by Date: RE: Standard uncertainties (SU) in the DDLm dictionary
- Prev by thread: Draft JSON specification for CIF
- Next by thread: Re: Draft JSON specification for CIF
- Index(es):