Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by mdiener21):

 Yes I do, the Shapefile origin is AutoCAD and this is where things get
 messy when converting to Shapefile then trying to import into PostGIS.  I
 have attached the problem Shapefile of multilinestrings  which is a false
 positive "linestring" according to GDAL and QGIS (which uses GDAL).  The
 Shapefile is of course valid with no geometry errors or invalid geometries
 all is good.  The problem is the Shapefile can store linestrings and
 multi-geom and still be valid.  When trying to import such a Shapefile
 PostGIS will NOT allow you to have both you must choose and in this case
 you must always choose the multi-geom "MULTILINESTRING" geom type
 otherwise boom error on import.

--
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/067.70a000d5dacdbf9424e5e3457014fd09%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by mdiener21):

 * Attachment "gas_lines.zip" added.

 Shapefile Example with both Linestrings and Multilinestrings

--
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/067.2d08c643cc1bc5859fcd00110f16902e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by claudep):

 Do you have an example of such a "deficient" layer? It would be nice to
 make a test case for this issue.

--
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/067.459f13345d04de260257e45ed3aafc34%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by mdiener21):

 Replying to [comment:5 claudep]:
 > I'm not convinced to force multi geometries by default. If you obtain
 errors when importing data from a shapefile because of a missing multi
 field, you can always improve your model and force the multi geometries in
 a later time. And if you don't want to risk the potential errors, there is
 the `multi_geom` parameter you can set yourself.
 >
 > Feel free to add more arguments if you think I missed the point.


 Please check out my comment :)  cheers

--
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/067.87af8e6963971931b9934284708fdd6c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by mdiener21):

 Hi the problem is this code if multi_geom and gtype.num in (1, 2, 3):
 LINE 213   gtype.num in (1,2,3)   does NOT always return TRUE for
 multi_geom because GDAL does not always guess correct.  Therefore if I set
 multi_geom to True it still fails because GDAL guesses wrong.  Another
 solution is to simply remove the GDAL check ?   any thoughts

--
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/067.e6ee4d4eb4b33a3f179a5f91c5f16083%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by claudep):

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


Comment:

 I'm not convinced to force multi geometries by default. If you obtain
 errors when importing data from a shapefile because of a missing multi
 field, you can always improve your model and force the multi geometries in
 a later time. And if you don't want to risk the potential errors, there is
 the `multi_geom` parameter you can set yourself.

 Feel free to add more arguments if you think I missed the point.

--
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/067.eae8245cc9633cb85fe7990a2f667c07%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #24207: Change ogrinspect multi_geom default to True (was: contrib gis change ogrinspect multi_geom default to True)

[Django]

#24207: Change ogrinspect multi_geom default to True
-+-
 Reporter:  mdiener21|Owner:  mdiener21
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  GIS  |  Version:  1.8alpha1
 Severity:  Normal   |   Resolution:
 Keywords:  gis, multi_geom  | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

--
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/067.4072862c52fd749947cd734fa3528c25%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.