Fatal Python error: deallocating None (during grid encoding) #1221

Closed
springmeyer opened this Issue May 18, 2012 · 7 comments

Comments

Projects
None yet
3 participants
@springmeyer
Member

springmeyer commented May 18, 2012

To support filtering truly null values from postgres (vs attributes not existing) this change was made to the postgis driver as part of #794.

To support this, in python, the value_converter returns Py_None, but this is a 'bare' Py_None will leak unless reference counted. Thanks to @migurski and @nvkelso for uncovering this bug.

@springmeyer

This comment has been minimized.

Show comment Hide comment
@springmeyer

springmeyer May 18, 2012

Member

fixed in master (future 2.1.x) in ebfbe79 and backported to 2.0.x (future 2.0.2) in 92dd305

Member

springmeyer commented May 18, 2012

fixed in master (future 2.1.x) in ebfbe79 and backported to 2.0.x (future 2.0.2) in 92dd305

springmeyer pushed a commit that referenced this issue May 18, 2012

@migurski

This comment has been minimized.

Show comment Hide comment
@migurski

migurski May 18, 2012

Member

So good, thank you!

Do you know what the release schedule for 2.0.2 on Ubuntu looks like? Is there one?

Member

migurski commented May 18, 2012

So good, thank you!

Do you know what the release schedule for 2.0.2 on Ubuntu looks like? Is there one?

@springmeyer

This comment has been minimized.

Show comment Hide comment
@springmeyer

springmeyer May 18, 2012

Member

I'll look for a moment to tag and release next week. Then we'd look for @dpaleino (of http://wiki.debian.org/DebianGis) to initiate debian packaging, as he's the official packager. Only after he has packaged for debian do the ubuntu guys follow.

Member

springmeyer commented May 18, 2012

I'll look for a moment to tag and release next week. Then we'd look for @dpaleino (of http://wiki.debian.org/DebianGis) to initiate debian packaging, as he's the official packager. Only after he has packaged for debian do the ubuntu guys follow.

@springmeyer

This comment has been minimized.

Show comment Hide comment
@springmeyer

springmeyer May 18, 2012

Member

And until then pull from the unofficial PPA I maintain at https://launchpad.net/~mapnik/+archive/nightly-2.0/

Member

springmeyer commented May 18, 2012

And until then pull from the unofficial PPA I maintain at https://launchpad.net/~mapnik/+archive/nightly-2.0/

@dpaleino

This comment has been minimized.

Show comment Hide comment
@dpaleino

dpaleino May 20, 2012

@springmeyer I was working right now for 2.0.1, since I received a mail from a user, and I totally missed that one release. Then I'll just wait and upload 2.0.2, if that's ok with you. Hoping that a nice and smooth transition is feasible :)

@springmeyer I was working right now for 2.0.1, since I received a mail from a user, and I totally missed that one release. Then I'll just wait and upload 2.0.2, if that's ok with you. Hoping that a nice and smooth transition is feasible :)

strk pushed a commit to strk/mapnik that referenced this issue Jun 12, 2012

@springmeyer

This comment has been minimized.

Show comment Hide comment
@springmeyer

springmeyer Jun 18, 2012

Member

@dpaleino - yes, going to look for time this week to tag 2.0.2.

Member

springmeyer commented Jun 18, 2012

@dpaleino - yes, going to look for time this week to tag 2.0.2.

@springmeyer

This comment has been minimized.

Show comment Hide comment
@springmeyer

springmeyer Aug 4, 2012

Member

@dpaleino - hey there, 2.0.2 is now officially out with this fix, please feel free to package: https://github.com/mapnik/mapnik/wiki/Release2.0.2

Member

springmeyer commented Aug 4, 2012

@dpaleino - hey there, 2.0.2 is now officially out with this fix, please feel free to package: https://github.com/mapnik/mapnik/wiki/Release2.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment