<div dir="ltr"><div dir="ltr">On Thu, Jun 13, 2019 at 11:42 AM Antony Lee <<a href="mailto:antony.lee@institutoptique.fr">antony.lee@institutoptique.fr</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi John,<br><br>Thanks for your writeup. It provides an interesting historical view on one of the oldest parts of the Matplotlib codebase, which is always nice read about.<br><br>It is great that you intend to work again on table.py </div></blockquote><div><br></div><div>I made no commitment to work on this again in the future ;)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">(which was mostly abandoned by the other core devs), but I am not sure to what extent the new API you propose has been stabilized, or whether you are still experimenting with it. There are also other issues with the API design (e.g. <a href="https://github.com/matplotlib/matplotlib/issues/12931" target="_blank">https://github.com/matplotlib/matplotlib/issues/12931</a>) that you may, perhaps, want to revisit?<br></div></blockquote><div><br></div><div>There are four (now 5) pull requests with various API changes / bug fixes for consideration -- just created one of issue 12931</div><div><br></div><div>Those requests themselves resulted in other discussion about API changes (eg should visible_edges change its name to border?) -- so some on-going experimentation.</div><div><br></div><div>Some of the bugs are such that they can't really be fixed without impacting the API.</div><div><br></div><div>There is a fair bit to discuss here -- most of the feedback that I have received so far has been about cosmetic stuff with very little comment on the substance of the changes.</div><div><br></div><div>From my point of view I am pretty much done experimenting at this point.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br>As you may have noticed, making backwards incompatible changes to Matplotlib is a bit of an onerous process (due to the deprecation process, the baseline images problem, etc.).</div></blockquote><div><br></div><div>Indeed. Which is a big reason these problems have languished so long.</div><div><br></div><div>Just because the process is onerous, doesn't mean it shouldn't be done.</div><div><br></div><div>The baseline_images problem feels like the tail wagging the dog.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"> Given that table.py is fairly self-contained, perhaps you may want to consider moving it to its own separate package?</div></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"> i.e. something published separately on PyPI (let us know if you need any help for that), so that one would do (for example)<br><br><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px">from mpltable import table<br>table(...)</blockquote><br>This way you can play with the API in whatever way you want, and we could just point to your package as the "new, more modern" way to make tables with Matplotlib, and possibly import it back to the main repo as a single PR once everything is stabilized.<br></div></blockquote><div><br></div><div>Whilst I can see the attraction with this approach, I am not really keen on it.</div><div><br></div><div>Based on my experiences so far trying to help out here, there is no guarantee that the new table would ever make it into the code and that would leave me with an increasing maintenance burden of pulling on-going changes to the code, whilst leaving matplotlib with a buggy and ugly table.</div><div><br></div><div>There are pros and cons with one big fix v a bunch of smaller ones. I lean towards the several smaller fix approach eg issue 12931 should probably be done as a standalone request, making it easier to back out later if it becomes necessary.</div><div><br></div><div>John</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br>Antony<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 13, 2019 at 5:01 PM <a href="mailto:swfiua@gmail.com" target="_blank">swfiua@gmail.com</a> <<a href="mailto:swfiua@gmail.com" target="_blank">swfiua@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>I just wanted to provide some background on the recent flurry of fixes to table.py and the history behind the code.</div><div><br></div><div>See attached pdf.</div><div><br></div><div>There are 3 main themes to the changes: </div><div><br></div><div>* better automatic setting of the font size</div><div>* change how horizontal padding around cell text is calculated</div><div>* add ability to specify cell edge colours</div><div><br></div><div>John</div></div>
_______________________________________________<br>
Matplotlib-devel mailing list<br>
<a href="mailto:Matplotlib-devel@python.org" target="_blank">Matplotlib-devel@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/matplotlib-devel</a><br>
</blockquote></div>
</blockquote></div></div>