PMXBOT Log file Viewer

Help | Karma | Search:

#pil logs for Tuesday the 1st of July, 2014

(Back to #pil overview) (Back to channel listing) (Animate logs)
[06:25:19] <travis-ci> [travis-ci] python-pillow/Pillow#1274 (master - 1b2dd37 : Hugo): The build passed.
[06:25:19] <travis-ci> [travis-ci] Change view : https://github.com/python-pillow/Pillow/compare/32a6d398d2a3...1b2dd37905f2
[06:25:19] <travis-ci> [travis-ci] Build details : http://travis-ci.org/python-pillow/Pillow/builds/28841841
[09:49:25] <aclark> wiredfool: do you want these last 3 PRs merged before release? 2 of them have test failures…
[10:46:35] <travis-ci> [travis-ci] python-pillow/Pillow#1279 (master - dae96f3 : Alex Clark): The build was broken.
[10:46:35] <travis-ci> [travis-ci] Change view : https://github.com/python-pillow/Pillow/compare/690f5639cdaa...dae96f37629d
[10:46:35] <travis-ci> [travis-ci] Build details : http://travis-ci.org/python-pillow/Pillow/builds/28857998
[11:02:50] <travis-ci> [travis-ci] python-pillow/Pillow#1280 (master - 492be8b : Alex Clark): The build was fixed.
[11:02:50] <travis-ci> [travis-ci] Change view : https://github.com/python-pillow/Pillow/compare/dae96f37629d...492be8ba3e75
[11:02:50] <travis-ci> [travis-ci] Build details : http://travis-ci.org/python-pillow/Pillow/builds/28859021
[13:56:22] <travis-ci> [travis-ci] hugovk/Pillow#362 (diff_cover_just_testing - a836741 : hugovk): The build passed.
[13:56:22] <travis-ci> [travis-ci] Change view : https://github.com/hugovk/Pillow/compare/71af64253d41...a83674166b8c
[13:56:22] <travis-ci> [travis-ci] Build details : http://travis-ci.org/hugovk/Pillow/builds/28871882
[15:50:16] <wiredfool> aclark: 754 is a go, 755 is waiting on debugging a fix to coveralls.
[15:51:19] <wiredfool> 737 has one issue, and either we way that on the balance it's better and open a bug, or we hold the whole thing
[15:52:08] <wiredfool> also, you are not in favor of the oxford comma. https://github.com/python-pillow/Pillow/commit/dae96f37629d41001498ece79b0e2eed7e1690ae
[16:18:53] <serp> when I use a ttf and try to draw it at (0,0) in an image it actually draws further down for some reasons... here is the code i use to reproduce... http://pastebin.com/raw.php?i=pcLdRGQx
[16:19:45] <serp> so in that example i resize the image to the size returned by textsize.. then draw the text at (0,0)... the bottom of the text gets cut off
[16:21:01] <wiredfool> IIRC, the start point is at a point that's consistent, either at the baseline or at the top of capitals.
[16:21:26] <wiredfool> It will extend past for fonts that have descenders or ascenders > cap height.
[16:21:52] <travis-ci> [travis-ci] wiredfool/Pillow#382 (terseus_imagedraw - e3f9fa0 : wiredfool): The build is still failing.
[16:21:52] <travis-ci> [travis-ci] Change view : https://github.com/wiredfool/Pillow/compare/668141a898c6...e3f9fa0d781f
[16:21:52] <travis-ci> [travis-ci] Build details : http://travis-ci.org/wiredfool/Pillow/builds/28884382
[16:23:06] <wiredfool> There is a way to get the text extents - I believe it's draw.textsize
[16:23:14] <wiredfool> serp ping
[16:23:15] <serp> wiredfool: well the text i used has capitals in the example text... and shouldn't textsize return the width and height?
[16:23:36] <serp> i use textsize in the example .. that's how i know the size to resize the image
[16:23:37] <wiredfool> hmm, you are using textsize
[16:24:07] <serp> it works as expected if i use the default font
[16:24:19] <serp> just when i switch to a ttf
[16:24:43] <serp> well the few ttf i've tested.. i've not done a lot of testing with other fonts
[16:24:44] <wiredfool> the default font is a bitmap, not using freetype
[16:24:53] <wiredfool> totally different code paths
[16:25:22] <serp> right but i would expect the same behaivor with textsize
[16:25:43] <wiredfool> no, totally different code paths. TTF fonts have metrics
[16:25:50] <wiredfool> default font has bounding boxes
[16:26:50] <wiredfool> you can look at font.getmetrics, or getsize
[16:27:02] <wiredfool> though they're badly documented as to what they actually return.
[16:28:44] <travis-ci> [travis-ci] wiredfool/Pillow#383 (multithread-testing-travis - 2ca9ffb : wiredfool): The build was fixed.
[16:28:44] <travis-ci> [travis-ci] Change view : https://github.com/wiredfool/Pillow/compare/9dd559cb50e2...2ca9ffba5d09
[16:28:44] <travis-ci> [travis-ci] Build details : http://travis-ci.org/wiredfool/Pillow/builds/28884566
[17:59:55] <18VAANB6P> [travis-ci] wiredfool/Pillow#387 (master - f8b6163 : Alex Clark): The build passed.
[17:59:55] <18VAANB6P> [travis-ci] Change view : https://github.com/wiredfool/Pillow/compare/bd9c555905af...a47b8c15da6a
[17:59:55] <18VAANB6P> [travis-ci] Build details : http://travis-ci.org/wiredfool/Pillow/builds/28891308
[18:03:00] <serp> wiredfool: for the record the following is a workaround... txt_image = txt_image.crop(txt_image.getbbox())
[18:05:49] <wiredfool> right, the bounding box would do that in that case
[18:18:24] <wiredfool> aclark: 755 doesn't work with coveralls yet
[18:19:35] <aclark> wiredfool: soooo… revert then?
[18:19:58] <wiredfool> only .travis.yml
[18:20:12] <wiredfool> I split that PR so that the rest of it sould go through
[18:20:12] <aclark> well let's just fix that then
[18:20:18] <aclark> aha
[18:22:47] <wiredfool> there's a bug in how python-coveralls is generating the coverage report when run with --nose-cov
[18:22:56] <wiredfool> coveage report works properly
[18:38:12] <travis-ci> [travis-ci] python-pillow/Pillow#1290 (master - febfe2f : Alex Clark ☺): The build was broken.
[18:38:12] <travis-ci> [travis-ci] Change view : https://github.com/python-pillow/Pillow/compare/a47b8c15da6a...febfe2f574a7
[18:38:12] <travis-ci> [travis-ci] Build details : http://travis-ci.org/python-pillow/Pillow/builds/28895982
[18:44:52] <travis-ci> [travis-ci] python-pillow/Pillow#1291 (master - b2ed31e : Alex Clark ☺): The build passed.
[18:44:52] <travis-ci> [travis-ci] Change view : https://github.com/python-pillow/Pillow/compare/febfe2f574a7...b2ed31e8cd13
[18:44:52] <travis-ci> [travis-ci] Build details : http://travis-ci.org/python-pillow/Pillow/builds/28895999
[19:48:19] <travis-ci> [travis-ci] hugovk/Pillow#363 (PalmImagePlugin - 3807af7 : hugovk): The build failed.
[19:48:19] <travis-ci> [travis-ci] Change view : https://github.com/hugovk/Pillow/compare/50bbb5e0c535^...3807af79e31a
[19:48:19] <travis-ci> [travis-ci] Build details : http://travis-ci.org/hugovk/Pillow/builds/28901102
[19:52:55] <aclark> wiredfool: is https://github.com/python-pillow/Pillow/pull/761 ready to go?
[19:53:44] <wiredfool> bah. there's a spare print.
[19:53:46] <wiredfool> one moment
[19:56:05] <wiredfool> right, now it is.
[19:57:13] <travis-ci> [travis-ci] hugovk/Pillow#364 (PalmImagePlugin - 7753071 : hugovk): The build passed.
[19:57:13] <travis-ci> [travis-ci] Change view : https://github.com/hugovk/Pillow/compare/3807af79e31a...775307113bf7
[19:57:13] <travis-ci> [travis-ci] Build details : http://travis-ci.org/hugovk/Pillow/builds/28902440
[22:38:40] <aclark> wiredfool: you don't want this merged as-is right? https://github.com/python-pillow/Pillow/pull/760
[22:43:56] <wiredfool> honestly, I don't know about 760.
[22:44:18] <aclark> wiredfool: k
[22:44:39] <wiredfool> since it's essentially a devtime patch, there's no need to get it in
[22:45:08] <aclark> right, skipping
[22:45:20] <wiredfool> I suppose it's about 30 seconds late
[22:45:35] <wiredfool> but can we freeze, nd get a signoff from cgohkle for windows?
[22:45:47] <wiredfool> so that we don't have a 2.5.1 due to windows?
[22:46:28] <aclark> Yeah I already asked him for one, I assume he would have raised by now
[22:46:51] <wiredfool> well, the imagedraw changes hit c code, and they were merged today
[22:47:21] <wiredfool> tho, I think the setup on travis now errors out on the sort of things that killed windows compiles, at least on 3.4
[23:11:29] <wiredfool> heh. /me went 7+ years with one tweet
[23:12:43] <wiredfool> but hell, I've ben dragged into the microsocialbloggingera
[23:15:53] <aclark> hah