Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes for records() speedup #67

Closed
wants to merge 4 commits into from
Closed

Fixes for records() speedup #67

wants to merge 4 commits into from

Commits on Aug 24, 2016

  1. Merge pull request #1 from GeospatialPython/master 

    Sync before fixing error
    @karimbahgat
    karimbahgat authored Aug 24, 2016
    Configuration menu
    Copy the full SHA
    af24202 View commit details
    Browse the repository at this point in the history

  2. Fix of records() bug 

    Previous records() speedup forgot that row values had to be parsed from
string into their appropriate field types. So moved the record parsing
into a separate method and call it where needed.

Also forgot that deleteflag is a field when grouping the flat rowlist
into rows. Changed to correct length.

All doctests pass again after these fixes.
    @karimbahgat
    karimbahgat committed Aug 24, 2016
    Configuration menu
    Copy the full SHA
    9220faf View commit details
    Browse the repository at this point in the history

  3. Fixed wrong doc for the __recordFmt() method

    @karimbahgat
    karimbahgat committed Aug 24, 2016
    Configuration menu
    Copy the full SHA
    0e88185 View commit details
    Browse the repository at this point in the history

  4. Fixed wrong doc for the __recordFmt() method

    @karimbahgat
    karimbahgat committed Aug 24, 2016
    Configuration menu
    Copy the full SHA
    2c8fd19 View commit details
    Browse the repository at this point in the history