Re: [Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
+--
 Reporter:  Carl Meyer  |Owner:  Michał Ociepka
 Type:  Bug |   Status:  closed
Component:  GIS |  Version:  1.10
 Severity:  Normal  |   Resolution:  wontfix
 Keywords:  py2 | Triage Stage:  Accepted
Has patch:  1   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  1
Easy pickings:  0   |UI/UX:  0
+--
Changes (by Tim Graham):

 * status:  assigned => closed
 * resolution:   => wontfix


Comment:

 From the person who worked on the pull request, "There is not elegant way
 to solve this issue. If Django drops support to python 3 it is pointless
 to continue."

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.279e7000f16c6d684067aeee524803a5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
+--
 Reporter:  Carl Meyer  |Owner:  Michał Ociepka
 Type:  Bug |   Status:  assigned
Component:  GIS |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  py2 | Triage Stage:  Accepted
Has patch:  1   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  1
Easy pickings:  0   |UI/UX:  0
+--
Changes (by Michał Ociepka):

 * needs_better_patch:  0 => 1
 * has_patch:  0 => 1


Comment:

 https://github.com/django/django/pull/7503

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.c99dbfbd105b9ccf0138e5d699c4992e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
+--
 Reporter:  Carl Meyer  |Owner:  Michał Ociepka
 Type:  Bug |   Status:  assigned
Component:  GIS |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  py2 | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--
Changes (by Michał Ociepka):

 * status:  new => assigned
 * owner:  nobody => Michał Ociepka


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.0e1f231c5b2271ce423ae4788b45ec4a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
+
 Reporter:  carljm  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  GIS |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  py2 | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+

Comment (by jtiai):

 Documentation
 
https://docs.djangoproject.com/en/1.10/ref/contrib/gis/geos/#django.contrib.gis.geos.GEOSGeometry.wkb
 at refers value type to be "Python buffer". Couldn't actually to find any
 examples where buffer is used as an input so wonder how many would be
 affected by the change.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.4eb0a37af37d39c44e76b770f14ed1e9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
+
 Reporter:  carljm  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  GIS |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  py2 | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+
Changes (by timgraham):

 * keywords:   => py2
 * stage:  Unreviewed => Accepted


Comment:

 Patches accepted by January 2017 at which point we're dropping Python 2
 support in master and could close this as wontfix.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.4909499eb2c195e1692926150516f2c0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #27088: GEOSGeometry should accept Python 2.7 memoryview

[Django]

#27088: GEOSGeometry should accept Python 2.7 memoryview
--+
   Reporter:  carljm  |  Owner:  nobody
   Type:  Bug | Status:  new
  Component:  GIS |Version:  1.10
   Severity:  Normal  |   Keywords:
   Triage Stage:  Unreviewed  |  Has patch:  0
Needs documentation:  0   |Needs tests:  0
Patch needs improvement:  0   |  Easy pickings:  0
  UI/UX:  0   |
--+
 Django's custom `django.util.six` module has its own `memoryview` alias
 which is `buffer` on Py2 and `memoryview` on Py3, and `GEOSGeometry`
 enforces that its input (if any kind of buffer) be an instance of
 `django.util.six.memoryview`
 
(https://github.com/django/django/blob/master/django/contrib/gis/geos/geometry.py#L73).

 However, Python 2.7 has backported the Py3 `memoryview` type, and its use
 is preferred over Py2 `buffer` for cross-compatible code:
 https://docs.python.org/2/c-api/buffer.html

 `GEOSGeometry` should accept Py2 `memoryview` instances.

 (For backwards-compatibility reasons, even though Python 2.7 is now the
 minimum supported version, we probably can't remove
 `django.util.six.memoryview` or change it to refer to `memoryview` on Py2,
 because Py2 `buffer` and `memoryview` don't have fully compatible APIs.)

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/049.e58281ce40003813ca569b5b6710c30c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.