Opened 6 years ago

Closed 5 years ago

#13267 closed Cleanup/optimization (fixed)

Update gis tutorial python examples

Reported by: samueladam Owned by: nobody
Component: Documentation Version: master
Severity: Normal Keywords: gis, tutorial, documentation
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Three minor fixes in documentation example:

  • import only DataSource instead of *
  • ogrinspect now returns 'geom' field instead of 'mpoly'
  • in the example verbose=verbose was not defined

Attachments (1)

gis_tutorial_update.diff (1.2 KB) - added by samueladam 6 years ago.
gis tutorial examples, minor fixes

Download all attachments as: .zip

Change History (9)

Changed 6 years ago by samueladam

Attachment: gis_tutorial_update.diff added

gis tutorial examples, minor fixes

comment:1 Changed 6 years ago by Russell Keith-Magee

Component: GISDocumentation
milestone: 1.2
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

comment:2 Changed 6 years ago by Russell Keith-Magee

Triage Stage: AcceptedReady for checkin

comment:3 Changed 6 years ago by Russell Keith-Magee

Patch needs improvement: set
Triage Stage: Ready for checkinAccepted

On closer inspection, I'm not convinced any of the three fixes are correct.

  • I don't know if we we want to encourage importing from gdal.datasource directly, or if gdal is the preferred location
  • The fix for 'geom' is inconsistenly applied - the following text talks about the mpoly option
    • verbose is defined; it's an argument to run.

comment:4 Changed 6 years ago by Russell Keith-Magee

milestone: 1.2

Deferring due to the absence of a trunk-ready patch.

comment:5 Changed 5 years ago by Julien Phalip

Severity: Normal
Type: Cleanup/optimization

comment:6 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:7 Changed 5 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:8 Changed 5 years ago by Claude Paroz

Resolution: fixed
Status: newclosed

In [17754]:

(The changeset message doesn't reference this ticket)

Note: See TracTickets for help on using tickets.
Back to Top