keep track of temporry elets added hen plotting#1449
Conversation
and removes them at clear time fix #1424
|
@aashish24 @dlonie @sankhesh please review |
|
@doutriaux1 is this some kind of alternate very efficient language which uses every 3rd letter? |
|
@durack1 what can I say? My brain goes so fast that my physical abilities just can't keep up 😜 |
|
@doutriaux1 do you want me to run the code indicated in #1424 to test this over a large number of loops? |
There was a problem hiding this comment.
These wouldn't pass the style checks. Let's try to use pep8 style in new commits (spaces between arguments/list items, 4 space indents, etc) -- less work to do later!
There was a problem hiding this comment.
please approve, I'll do the pep8 anyway and autopep8 does this... No time for this right now. Thanks.
There was a problem hiding this comment.
@doutriaux1 if this is urgent, I would suggest you pass the branch name to the folks that need it (or create staging). I think it would be better if we take care of the code at the review level.
There was a problem hiding this comment.
let's let it rot here then... I just can't get to it now, it fixes memory growth issue. I still need to run pep8 fix on vcs anyway, I don't want to it twice...
There was a problem hiding this comment.
when I do the vcs pep8, then I will rebase this on top of it and do the pep8 on this too. You guys can approve it then. That's fine with me.
|
@doutriaux1 let's merge it then. If are planning to take care of vcs style issues soon. 👍 |
keep track of temporry elets added hen plotting
|
Thanks @aashish24 as soon as I reviewed and merged @dlonie PR I will flake8 it. |
4 participants
and removes them at clear time
fix #1424