IETF Logo Mail Archive

Re: [Geojson] Ben Campbell's No Objection on draft-ietf-geojson-03: (with COMMENT)

Tim Schaub <tim.schaub@gmail.com> Thu, 02 June 2016 06:06 UTC

Return-Path: <tim.schaub@gmail.com>
X-Original-To: geojson@ietfa.amsl.com
Delivered-To: geojson@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AE8FF12D1AB; Wed, 1 Jun 2016 23:06:52 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id lJElM3MtgOy2; Wed, 1 Jun 2016 23:06:50 -0700 (PDT)
Received: from mail-io0-x229.google.com (mail-io0-x229.google.com [IPv6:2607:f8b0:4001:c06::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 299D312D166; Wed, 1 Jun 2016 23:06:50 -0700 (PDT)
Received: by mail-io0-x229.google.com with SMTP id t40so37649794ioi.0; Wed, 01 Jun 2016 23:06:50 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=CDHPCzKk7APC6+pF+4Xcb7wWGoghJSuv4Aw1qSYqRG0=; b=s2hldukdYJ+dLkfi1b+voVBAiXsQquH8Gr3jLGuX740O+oPZzZXcOlFMxRBAkmQX8d svSpFbos/gsP1BFtOwUaKATa5A3IswPFfaJiDa7XKmoM9rG1JWbBvoS4Y/QEQ7m9DF8l oLBx9DBadKes4N0MIo+VJBZTXTMwlC0FzdrjmbeN1xG+Q57z4u+RlIHDpG5GOMyRuxfp vRTotjm4D1G2/k5nq8OL8DpQoH1WJcMB/wi3t+fWqcohvY7bojx27yoRgQwBvaoahAe3 XrGphsF9yrN3UCj2SZkF+nu4+wtx8ZKE5y64lNuzmmTw+siSbz1Lan87bvQXxWdZfaDC h4oQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=CDHPCzKk7APC6+pF+4Xcb7wWGoghJSuv4Aw1qSYqRG0=; b=kU6WIe2FqPNOMsf28uQOHYWcRoqTBz3E9nEoUHMZBnbFehuiQBF/2VQoNnBSMzRkh7 GC6MdkqUdwG9p8QI2ZvFo5M0+kF2Z1sgsUtUCSzx6UhWnngTQ97uNCimXTLFTxLRf8sr w1pIou1PbWuWe8MgEmN2SFb7qHePV2c33slIcKKKqcRV0HQCOu/DsWo/j7xriR7E2d/A 5KrTQjlqeox6X4Q5icTTlh+IjdRa/Z018ZOETuzBR8co3A6JEDF13aXbGXtW01RHRzOD n0GW+xucXOQtY3mw1Xf35koNkvFnN9MHgtdo6a5n7tk61TQm4WKE36NowpKQCNz60WmJ cBZA==
X-Gm-Message-State: ALyK8tJSPPGZ94/K4/RwBLtmOVfsUXJH0jRKZWpxArIzo2p38vU0h5wNG3mgP91+z5qQJ29UkZXEknaz77utpg==
MIME-Version: 1.0
X-Received: by 10.107.15.23 with SMTP id x23mr1519571ioi.36.1464847609411; Wed, 01 Jun 2016 23:06:49 -0700 (PDT)
Received: by 10.64.252.197 with HTTP; Wed, 1 Jun 2016 23:06:49 -0700 (PDT)
In-Reply-To: <07CC6F45-4E9A-4A18-8387-5519DE361D22@nostrum.com>
References: <20160602020613.16111.53924.idtracker@ietfa.amsl.com> <CAKdrn+dAiUqAx5EOX3xdW4PoYSVZ=KJGfdMgC2fz=7k0UCbENA@mail.gmail.com> <07CC6F45-4E9A-4A18-8387-5519DE361D22@nostrum.com>
Date: Wed, 1 Jun 2016 23:06:49 -0700
Message-ID: <CAKdrn+eRsKsimka2xN_JqqgiSy7M1U99z83vMO0t1okToYVMAw@mail.gmail.com>
From: Tim Schaub <tim.schaub@gmail.com>
To: Ben Campbell <ben@nostrum.com>
Content-Type: text/plain; charset=UTF-8
Archived-At: <http://mailarchive.ietf.org/arch/msg/geojson/gwVsFisIXRVfSiUsuDCHJ4vIFxg>
Cc: draft-ietf-geojson@ietf.org, geojson-chairs@ietf.org, The IESG <iesg@ietf.org>, geojson@ietf.org
Subject: Re: [Geojson] Ben Campbell's No Objection on draft-ietf-geojson-03: (with COMMENT)
X-BeenThere: geojson@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF GeoJSON WG <geojson.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/geojson>, <mailto:geojson-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/geojson/>
List-Post: <mailto:geojson@ietf.org>
List-Help: <mailto:geojson-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/geojson>, <mailto:geojson-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 02 Jun 2016 06:06:53 -0000

On Wed, Jun 1, 2016 at 9:57 PM, Ben Campbell <ben@nostrum.com> wrote:
> Thanks for the response. Comments inline:
>
> On 2 Jun 2016, at 0:49, Tim Schaub wrote:
>
>> Thanks for the comments Ben.  My thoughts on one of them below ...
>>
>> On Wed, Jun 1, 2016 at 7:06 PM, Ben Campbell <ben@nostrum.com> wrote:
>>>
>>>
>>> ----------------------------------------------------------------------
>>> COMMENT:
>>> ----------------------------------------------------------------------
>>>
>>
>> ...
>>
>>> - 3.1.6, 4th bullet: Why SHOULD? Can you imagine situations where it
>>> would be reasonable to not follow the right-hand rule?
>>
>>
>> There has been an effort throughout the IETF process to reduce the
>> likelihood that GeoJSON in use today (pre-IETF GeoJSON) will not be
>> rejected as invalid by strict-parsers developed in the future.  The
>> GJ2008 document didn't discuss winding order of rings.  So while it
>> would be convenient for consumers if they didn't have to check the
>> winding order (i.e. if the spec said MUST follow the right-hand rule),
>> I think this is a recommendation only in hopes of improving
>> interoperability while retaining some backwards compatibility.
>
>
> That's a reasonable motivation for a SHOULD. I'd be fine if you left it htat
> way. A few words of explanation to that effect might be useful.
>
>>
>> I wouldn't be opposed to making this MUST.  People will write and use
>> lenient parsers until old (left-hand) GeoJSON falls out of use.  With
>> SHOULD, we'll have mixed-handedness forever.
>
>
> Another possible approach would be to say that implementations of this spec
> MUST use right-hand windings, but that parsers SHOULD not reject left-handed
> windings for reasons of backwards compatibility.

I like this idea.  Here is a pull request suggesting changes along
these lines: https://github.com/geojson/draft-geojson/pull/207

I added a note about GJ2008 compatibility.  See
https://circle-artifacts.com/gh/geojson/draft-geojson/143/artifacts/0/home/ubuntu/draft-geojson/draft.html#rfc.section.3.1.6

Curious to hear what others think.

Tim

> Thanks!
>
> Ben.

v2.8.7 | Report a Bug | By Email