-
Notifications
You must be signed in to change notification settings - Fork 751
memoryleak for float #146
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.
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
Comments
@roarlarsen thank you for spotting this, so do you mean something like this: |
hi, I didn't check out the develop version, only the master, so I see the line 581 I refer to should have been~688, the leak is for float type, the decref is missing.But, the code you suggest is an possible memleak as well in case the char Roar LarsenTORDIVEL AS E-mail : [email protected] Web : http://www.tordivel.no...... THE VISION SOFTWARE HOUSEOn 11 February 2016 at 01:22, denfromufa [email protected] wrote:
|
few more observations in develop version..... there are several IFDEF's PYTHONnn where there seems to be a decref missing line #653 - int err = Runtime.PyObject_Compare(check, op); the op object is decref'ed at line #647 - small risk of GC deletion op Roar LarsenTORDIVEL AS E-mail : [email protected] Web : http://www.tordivel.no...... THE VISION SOFTWARE HOUSEOn 11 February 2016 at 08:35, Roar Larsen [email protected] wrote:
|
@roarlarsen @tonyroberts thanks again, do you agree with these changes: |
Looks fine What is current status of this project, don't see too many references to Roar Larsen
|
There is python.net tag on stackoverflow What else do you think is missing? can you share minimal code with delphi, how is it related to .NET? On Thursday, February 11, 2016, Roar Larsen [email protected]
|
forgot to mention ironpython docs and even book which are mostly applicable On Thursday, February 11, 2016, Denis Akhiyarov [email protected]
|
I'm aware of the stackoverflow thread, but I miss some more python samples We have a native delphi application with an embedded python engine so we We are machine vision company, and we're using opencv, numpy, scipi and Roar LarsenTORDIVEL AS E-mail : [email protected] Web : http://www.tordivel.no...... THE VISION SOFTWARE HOUSEOn 12 February 2016 at 04:06, denfromufa [email protected] wrote:
|
@roarlarsen I assume you have seen this tutorial: http://pythonnet.github.io/readme.html also this ironpython tutorial: http://ironpython.net/documentation/dotnet/ Perhaps someone can turn this into ipython notebook. |
detected memory leak in converter.cs - ToPrimitive function
missing Runtime.Decref(op); at line 581
The text was updated successfully, but these errors were encountered: