[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1329 Looks great to me, thank you @agoodm! Will merge to master later today, if there is no further discussion --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1329 @bzz Done! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if th

[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1329 Thank you for prompt and thoughtful comparison @agoodm ! > the choice of default format will matter in terms of backwards compatibility as the change should be seamless for users This

[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1329 @bzz The way I see it the pros and cons of each format can be summed up as follows: ### SVG Pros: - Vector so image quality will be consistent regardless of resolution - Displa

[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1329 @agoodm thank you for improving matplotlib integration! Please correct me, but right now PR does 2 things: - add PNG support, as well as SVG - switches default to PNG I see y

[GitHub] zeppelin issue #1329: ZEPPELIN-1318 - Add support for matplotlib displaying ...

2016-08-12 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1329 @felixcheung Done. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,