marker color handling

Hi Eric, hi everybody,

I just noticed that you, Eric, have committed r2799 to SVN which partly
reverts my commit r2790.

I understand that, obviously, the changes that I introduced are more
controversial than I would have expected, so we should try to reach an
agreement.

Was your objection generally about my patch or just about some detail? I
know that the patch might have broken a bit of compatibility, but since
the old logic was inconsistent and hard to understand, I figured that a
cleaner and simpler logic would justify a some possible break of
compatibility.

If you disagree with some change in principle, I would like to know your
objection. If there simply was a bug introduced by the change, I
appologize and hope I can fix it.

In any case, I have to point out that the current state of the SVN code
after the partial reversion of my commit is not self-consistent: My
commit changed several files besides axes.py - only reverting this one
file is bound to cause problems.

Greetings,
Norbert

Norbert,

Your 2790 patch seems to have completely wiped out the handling of marker colors in the simple case "N.plot(N.random.random(1000),'r.')", as noted by Stefan Van der Walt in a message yesterday to the matplotlib users list. I presume you are monitoring that list, and saw his message and my reply in which I noted that I was reverting the changes to axes.py. I looked at the changes to axes.py, and it appeared that you had simply forgotten the marker color handling; otherwise, the changes appeared to be code rearrangements, not API changes that would interact with other files, so I thought that reverting that file alone would work--that is, restore basic functionality to the svn version. My original thought was to wait and let you fix it, and evidently I should have, but I decided it would be better not to leave svn without the marker color functionality in the interim, and I had no way of knowing whether you would be able to get to it right away. In any case, I made the change, verified that it did restore marker color handling, committed it to svn, and left it at that.

Of course I was too hasty, and neglected to look at the patch as a whole; now I see that you were trying to shift the setting of marker colors out of axes.py and into lines.py. I still don't know why Stefan's simple example was broken, and I don't have time now to understand your patch properly, though I wish I did. I certainly don't object to the idea of trying to make the code cleaner and easier to follow.

So: please either restore your patch with additional changes to fix the problem Stefan found, or revert all of it cleanly until this can be sorted out.

If you think there might be incompatibilities introduced by your patch, then it would be good to know what they are.

Sorry to have made the situation more confusing than necessary.

Eric

Norbert Nemec wrote:

···

Hi Eric, hi everybody,

I just noticed that you, Eric, have committed r2799 to SVN which partly
reverts my commit r2790.

I understand that, obviously, the changes that I introduced are more
controversial than I would have expected, so we should try to reach an
agreement.

Was your objection generally about my patch or just about some detail? I
know that the patch might have broken a bit of compatibility, but since
the old logic was inconsistent and hard to understand, I figured that a
cleaner and simpler logic would justify a some possible break of
compatibility.

If you disagree with some change in principle, I would like to know your
objection. If there simply was a bug introduced by the change, I
appologize and hope I can fix it.

In any case, I have to point out that the current state of the SVN code
after the partial reversion of my commit is not self-consistent: My
commit changed several files besides axes.py - only reverting this one
file is bound to cause problems.

Greetings,
Norbert

OK, I understand. I will try sort out the problem today or otherwise
take back the whole patch.

I had not seen the message in matplotlib-users because of the backlog of
400 messages that have kept piling up...

Don't worry about additional confusion. Quickly reverting the patch was
the best thing to do for the moment.

Greetings,
Norbert

Eric Firing wrote:

···

Norbert,

Your 2790 patch seems to have completely wiped out the handling of
marker colors in the simple case "N.plot(N.random.random(1000),'r.')",
as noted by Stefan Van der Walt in a message yesterday to the matplotlib
users list. I presume you are monitoring that list, and saw his message
and my reply in which I noted that I was reverting the changes to
axes.py. I looked at the changes to axes.py, and it appeared that you
had simply forgotten the marker color handling; otherwise, the changes
appeared to be code rearrangements, not API changes that would interact
with other files, so I thought that reverting that file alone would
work--that is, restore basic functionality to the svn version. My
original thought was to wait and let you fix it, and evidently I should
have, but I decided it would be better not to leave svn without the
marker color functionality in the interim, and I had no way of knowing
whether you would be able to get to it right away. In any case, I made
the change, verified that it did restore marker color handling,
committed it to svn, and left it at that.

Of course I was too hasty, and neglected to look at the patch as a
whole; now I see that you were trying to shift the setting of marker
colors out of axes.py and into lines.py. I still don't know why
Stefan's simple example was broken, and I don't have time now to
understand your patch properly, though I wish I did. I certainly don't
object to the idea of trying to make the code cleaner and easier to follow.

So: please either restore your patch with additional changes to fix the
problem Stefan found, or revert all of it cleanly until this can be
sorted out.

If you think there might be incompatibilities introduced by your patch,
then it would be good to know what they are.

Sorry to have made the situation more confusing than necessary.

Eric

Norbert Nemec wrote:
  

Hi Eric, hi everybody,

I just noticed that you, Eric, have committed r2799 to SVN which partly
reverts my commit r2790.

I understand that, obviously, the changes that I introduced are more
controversial than I would have expected, so we should try to reach an
agreement.

Was your objection generally about my patch or just about some detail? I
know that the patch might have broken a bit of compatibility, but since
the old logic was inconsistent and hard to understand, I figured that a
cleaner and simpler logic would justify a some possible break of
compatibility.

If you disagree with some change in principle, I would like to know your
objection. If there simply was a bug introduced by the change, I
appologize and hope I can fix it.

In any case, I have to point out that the current state of the SVN code
after the partial reversion of my commit is not self-consistent: My
commit changed several files besides axes.py - only reverting this one
file is bound to cause problems.

Greetings,
Norbert
    
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel