Re: Draft JSON specification for CIF
- Subject: Re: Draft JSON specification for CIF
- From: James Hester <jamesrhester@xxxxxxxxx>
- Date: Thu, 13 Apr 2017 17:29:02 +1000
- DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;h=mime-version:in-reply-to:references:from:date:message-id:subject:to;bh=MHGt83vIlBSnr3IuAlMcVvjMU3DsDvK/RFXtoLXfHGM=;b=BEsEmkbWIlRQweORR3YpApPcFCpHS+sb6C/MdEjmeI9WppsLixGU1+YXUMdNdjidkvK33s8tqweCXHgSU9ZyOJ5/IGDVZu1Nkp+FvG74oKWJ0MjZ9BMVLWEsb7d1mo7JSynWawhCEp3S10fTaI3QIZB5X4eW9dE6hKmnIjujTWJ79CsjYuTt3MpNKM9AzMFaAPMHDaytF9YA9mkcl6XBLqRGdQvrBGv6LzirTMBQ7PlhHABMMoBENi/v9TCJUjv+pjyrhPh+G/64qlC6mmNJf9/1GBGyZFsB7LVPUYt43uo0jxcmRoWTQmjsEMJXSAODaei6jgiU1yuhvzvFOeK/yQ==
- In-Reply-To: <DM5PR04MB05083823C1CEDFD693DD1638E0030@DM5PR04MB0508.namprd04.prod.outlook.com>
- References: <CAM+dB2fszww=4A_w6evqg=5O9KKLnujajmg_SPSX=hCRQiBPtg@mail.gmail.com><DM5PR04MB05083823C1CEDFD693DD1638E0030@DM5PR04MB0508.namprd04.prod.outlook.com>
Dear CIF-developers,
A JSON representation of CIF is absolutely something that can and should be done, and the proposal seems viable and fairly complete. Nevertheless, there are some aspects of the proposal that bear discussion:
1. Which JSON are we talking about? The one described within ECMA-262? The one described by ECMA-404? Presumably not RFC4627, but maybe its successor, RFC7159? In fact, my vote would be for I-JSON, as described by RFC7493 (a restricted profile of RFC7159 JSON). I-JSON handles several interoperability issues that otherwise we would either suffer or have to handle explicitly.
2. It would be useful to explicitly consider which CIF structural and semantic constraints are to be mapped to JSON language constraints. In particular,
a) The proposed representation may inherently enforce the uniqueness of data names within a data block / save frame (supposing that we’re using I-JSON; see also below), but it does not inherently enforce the constraint that every data name in the same loop has the same number of values. I raise these together because it would be relatively straightforward to flip that, so that it is having the same number of values that is inherently enforced. On the other hand, I think there are alternatives that would allow both to be enforced, at the cost of a more complex and / or opaque structure. (I reserve further comment on such alternatives, pending interest from the group.)
b) CIF requires not just that block names, frame names, and data names be unique within their respective scopes, but that their *normalized forms* be unique within their scopes. That stronger constraint is alien to JSON; do we care about JSON-CIF enforcing that as an (I-)JSON constraint?
I like Bob Hansen's suggestion that all datanames be lower-cased, which becomes in the CIF2 context normalised form. This would seem to involve minimal effort on the part of CIF2->JSON converters.
c) I don’t see a particular value in placing stronger character encoding constraints on JSON-CIF than the underlying JSON standard places. Selection of I-JSON would make this moot, however, for I-JSON in fact requires UTF-8 encoding, just as is proposed for JSON-CIF.
3. We have again run into the CIF quirk of having two distinct null values, whereas JSON has only one. Although the two-character string "\\?" is certainly reminiscent of the special CIF ? value (as "\\." would be reminiscent of the . value), why not instead choose a value such as "\u0001" or "\uFFFF", which will never appear in a conforming instance of CIF’s native serialization?
4. The aspect of the proposal that I like least is the separate table of uncertainties. CIF data conforming to a dictionary, such as mmCIF, that provides separate data names for SUs does not need it, and I don’t immediately see why the native serialization format (presented as a string) is not a suitable choice for uncertainty-bearing values of items without a separate SU item.
--
_______________________________________________ cif-developers mailing list cif-developers@iucr.org http://mailman.iucr.org/cgi-bin/mailman/listinfo/cif-developers
Reply to: [list | sender only]
- References:
- Draft JSON specification for CIF (James Hester)
- RE: Draft JSON specification for CIF (Bollinger, John C)
- Prev by Date: Re: Draft JSON specification for CIF
- Next by Date: Re: Draft JSON specification for CIF
- Prev by thread: Re: Draft JSON specification for CIF
- Next by thread: Re: Draft JSON specification for CIF
- Index(es):