{"action":"created","issue":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/4525","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/4525/labels{/name}","comments_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/4525/comments","events_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/4525/events","html_url":"https://github.com/matplotlib/matplotlib/issues/4525","id":88256473,"number":4525,"title":"masked arrays broken in py3k + gcc 5.1 on arch linux","user":{"login":"tacaswell","id":199813,"avatar_url":"https://avatars.githubusercontent.com/u/199813?v=3","gravatar_id":"","url":"https://api.github.com/users/tacaswell","html_url":"https://github.com/tacaswell","followers_url":"https://api.github.com/users/tacaswell/followers","following_url":"https://api.github.com/users/tacaswell/following{/other_user}","gists_url":"https://api.github.com/users/tacaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/tacaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tacaswell/subscriptions","organizations_url":"https://api.github.com/users/tacaswell/orgs","repos_url":"https://api.github.com/users/tacaswell/repos","events_url":"https://api.github.com/users/tacaswell/events{/privacy}","received_events_url":"https://api.github.com/users/tacaswell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/build","name":"build","color":"fad8c7"},{"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/needs_review","name":"needs_review","color":"009800"},{"url":"https://api.github.com/repos/matplotlib/matplotlib/labels/Release%20critical","name":"Release critical","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"mdboom","id":38294,"avatar_url":"https://avatars.githubusercontent.com/u/38294?v=3","gravatar_id":"","url":"https://api.github.com/users/mdboom","html_url":"https://github.com/mdboom","followers_url":"https://api.github.com/users/mdboom/followers","following_url":"https://api.github.com/users/mdboom/following{/other_user}","gists_url":"https://api.github.com/users/mdboom/gists{/gist_id}","starred_url":"https://api.github.com/users/mdboom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdboom/subscriptions","organizations_url":"https://api.github.com/users/mdboom/orgs","repos_url":"https://api.github.com/users/mdboom/repos","events_url":"https://api.github.com/users/mdboom/events{/privacy}","received_events_url":"https://api.github.com/users/mdboom/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/matplotlib/matplotlib/milestones/18","html_url":"https://github.com/matplotlib/matplotlib/milestones/next%20point%20release","labels_url":"https://api.github.com/repos/matplotlib/matplotlib/milestones/18/labels","id":970105,"number":18,"title":"next point release","description":"Next minor release.","creator":{"login":"tacaswell","id":199813,"avatar_url":"https://avatars.githubusercontent.com/u/199813?v=3","gravatar_id":"","url":"https://api.github.com/users/tacaswell","html_url":"https://github.com/tacaswell","followers_url":"https://api.github.com/users/tacaswell/followers","following_url":"https://api.github.com/users/tacaswell/following{/other_user}","gists_url":"https://api.github.com/users/tacaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/tacaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tacaswell/subscriptions","organizations_url":"https://api.github.com/users/tacaswell/orgs","repos_url":"https://api.github.com/users/tacaswell/repos","events_url":"https://api.github.com/users/tacaswell/events{/privacy}","received_events_url":"https://api.github.com/users/tacaswell/received_events","type":"User","site_admin":false},"open_issues":209,"closed_issues":163,"state":"open","created_at":"2015-02-07T21:22:30Z","updated_at":"2015-06-15T14:45:50Z","due_on":"2015-07-06T04:00:00Z","closed_at":null},"comments":9,"created_at":"2015-06-14T22:11:34Z","updated_at":"2015-06-15T15:05:03Z","closed_at":null,"body":"This demo http://matplotlib.org/devdocs/examples/pylab_examples/color_by_yvalue.html is broken on python 3.\r\n\r\nOnly the left most plotted region is shown and happens either with ma arrays or `np.nan` filled arrays (have not started digging yet, I assume the eventually follow the same code path).  Filling with 0 works as expected.\r\n\r\nI think bisect on this goes back to the cxx removal but many of the commits in there don't compile on python3 due to a numpy related gcc error so that isn't very helpful.\r\n\r\nThis smells like a subtle clipping bug.\r\n\r\nattn @mdboom @efiring \r\n\r\nThe most common compilation error is:\r\n\r\n```python\r\ngcc -pthread -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -DPY_ARRAY_UNIQUE_SYMBOL=MPL_matplotlib_ft2font_ARRAY_API -DNPY_NO_DEPRECATED_API=NPY_1_7_API_VERSION -I/home/tcaswell/.virtualenvs/dd_3k/lib/python3.4/site-packages/numpy/core/include -I/home/tcaswell/.virtualenvs/dd_3k/include/freetype2 -I/usr/local/include -I/usr/include -I. -I/home/tcaswell/.virtualenvs/dd_3k/include/python3.4m -c src/ft2font_wrapper.cpp -o build/temp.linux-x86_64-3.4/src/ft2font_wrapper.o\r\nIn file included from src/file_compat.h:7:0,\r\n                 from src/ft2font_wrapper.cpp:3:\r\nsrc/ft2font_wrapper.cpp: In function ‘PyObject* PyFT2Image_as_array(PyFT2Image*, PyObject*, PyObject*)’:\r\nsrc/ft2font_wrapper.cpp:142:47: error: ‘PyArray_UBYTE’ was not declared in this scope\r\n     return PyArray_SimpleNewFromData(2, dims, PyArray_UBYTE, self->x->get_buffer());\r\n                                               ^\r\n/home/tcaswell/.virtualenvs/dd_3k/lib/python3.4/site-packages/numpy/core/include/numpy/ndarrayobject.h:129:46: note: in definition of macro ‘PyArray_SimpleNewFromData’\r\n         PyArray_New(&PyArray_Type, nd, dims, typenum, NULL, \\\r\n                                              ^\r\n\r\n```\r\n\r\nGoing to try rolling back to older versions of numpy te see if that fixes it.\r\n\r\nFor future reference, the output of bisect:\r\n\r\n\r\nThe first bad commit could be any of:\r\n\r\n4a56cdadeb874fd959263fdb7af474c2416457f4\r\ncfbf8d5225ac696cb2c3a8ae096911470074a5f8\r\n339537d58374e8c2f7dd62299a9869d819956937\r\n8dae8186c82f7243b819162d6eb1a64c1afeb379\r\n6e19dac792a42e6cc0f68f2066248a35657673ff\r\na22596839a7eb31d9700822f2ae09f43c2b765ce\r\n697b3ef37c044787b01344ca166a24d4da58345b\r\nc5ab7474a02f16e75e1bbabcb7af5f7d72f6f847\r\n5204635911f911a48c03ca3cf2f1f70341d396cf\r\naf578f5f35b2d7fef36a35a50d156c027472a816\r\n0624664f857f35583301f55553d28e00a4013770\r\n6d0717930000cafeeecabed900ac1b286a4176da\r\nba4016014cb4fb4927e36ce8ea429fed47dcb787\r\nd862c4740984433545461ded596d421f48f6b6a7\r\nd8c1d0d5e494465cbc27a639d5337240d84997fe\r\n\r\nWe cannot bisect more!\r\n\r\nBisect Log:\r\ngit bisect start 'master' 'v1.4.3'\r\na165735 bad Merge pull request #4521 from jkseppan/issue-4475\r\n21f46ff good REL : version 1.4.3\r\n\r\ngit bisect bad 056ba7a87e5bbd37dbf61a3c27329dc4f5648a54\r\n056ba7a bad MNT : Remove ``NavigationToolbar2QTAgg``\r\n\r\ngit bisect bad 112f199e26a0ad2047e626f4ceea437196a75909\r\n112f199 bad Merge pull request #3752 from astrofrog/rc_context_exceptions\r\n\r\ngit bisect good 34ed45af6c693d6951a82619bb669fc5463f4f9d\r\n34ed45a good Merge pull request #3291 from joferkington/lightsource-enhancements\r\n\r\ngit bisect bad c6f24d3edc787303d631d62c74893bf9bcaaf152\r\nc6f24d3 bad Merge pull request #3665 from jkseppan/fix-gdk-strides\r\n\r\ngit bisect good 0f4a68d3b75ae5ef192b1c90d876e18ff9de86c5\r\n0f4a68d good Merge pull request #3547 from jkseppan/numpy-deprecation\r\n\r\ngit bisect bad 7891790ea379d5960fbd61f4667919519456a9a4\r\n7891790 bad Remove reference to PyCXX\r\n\r\ngit bisect skip 697b3ef37c044787b01344ca166a24d4da58345b\r\n697b3ef skip Restore parallel testing\r\n\r\ngit bisect skip 6d0717930000cafeeecabed900ac1b286a4176da\r\n6d07179 skip Change how array converter functions are defined\r\n\r\ngit bisect good 7b206eb0eb150c9b3f8f451977055500c12d92ed\r\n7b206eb good Skip shading of masked array test on numpy 1.6\r\n\r\ngit bisect skip 8dae8186c82f7243b819162d6eb1a64c1afeb379\r\n8dae818 skip Remove some obsolete TODOs\r\n\r\ngit bisect skip af578f5f35b2d7fef36a35a50d156c027472a816\r\naf578f5 skip Add .clang-format to repository\r\n\r\ngit bisect good ea8617d01daa63201d026688efd994423c54659c\r\nea8617d good fix E201 and E202 pep8 errors\r\n\r\ngit bisect skip 4a56cdadeb874fd959263fdb7af474c2416457f4\r\n4a56cda skip Fix compilation\r\n\r\ngit bisect skip d862c4740984433545461ded596d421f48f6b6a7\r\nd862c47 skip Fix Mac compile again\r\n\r\ngit bisect skip cfbf8d5225ac696cb2c3a8ae096911470074a5f8\r\ncfbf8d5 skip Remove debugging prints\r\n\r\ngit bisect skip ba4016014cb4fb4927e36ce8ea429fed47dcb787\r\nba40160 skip Remove use of PyCXX in core C++ extensions\r\n\r\ngit bisect skip 339537d58374e8c2f7dd62299a9869d819956937\r\n339537d skip Fix curve simplification\r\n\r\ngit bisect good dbeed94d578c458d31842cd13c04f665155986cb\r\ndbeed94 good Merge pull request #3661 from jenshnielsen/numpy_16_fixes\r\n\r\ngit bisect skip 6e19dac792a42e6cc0f68f2066248a35657673ff\r\n6e19dac skip Fix segfault on Python 3\r\n\r\ngit bisect skip 0624664f857f35583301f55553d28e00a4013770\r\n0624664 skip Fix test_hexbin_pickable test\r\n\r\ngit bisect skip c5ab7474a02f16e75e1bbabcb7af5f7d72f6f847\r\nc5ab747 skip More Python 3 fixes\r\n\r\ngit bisect skip a22596839a7eb31d9700822f2ae09f43c2b765ce\r\na225968 skip Documentation\r\n\r\ngit bisect skip 5204635911f911a48c03ca3cf2f1f70341d396cf\r\n5204635 skip Windows-related fixes, thanks to @cgohlke\r\n\r\ngit bisect bad d8c1d0d5e494465cbc27a639d5337240d84997fe\r\nd8c1d0d bad Remove usage of old Numpy API\r\n"},"comment":{"url":"https://api.github.com/repos/matplotlib/matplotlib/issues/comments/112099760","html_url":"https://github.com/matplotlib/matplotlib/issues/4525#issuecomment-112099760","issue_url":"https://api.github.com/repos/matplotlib/matplotlib/issues/4525","id":112099760,"user":{"login":"jenshnielsen","id":548266,"avatar_url":"https://avatars.githubusercontent.com/u/548266?v=3","gravatar_id":"","url":"https://api.github.com/users/jenshnielsen","html_url":"https://github.com/jenshnielsen","followers_url":"https://api.github.com/users/jenshnielsen/followers","following_url":"https://api.github.com/users/jenshnielsen/following{/other_user}","gists_url":"https://api.github.com/users/jenshnielsen/gists{/gist_id}","starred_url":"https://api.github.com/users/jenshnielsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenshnielsen/subscriptions","organizations_url":"https://api.github.com/users/jenshnielsen/orgs","repos_url":"https://api.github.com/users/jenshnielsen/repos","events_url":"https://api.github.com/users/jenshnielsen/events{/privacy}","received_events_url":"https://api.github.com/users/jenshnielsen/received_events","type":"User","site_admin":false},"created_at":"2015-06-15T15:05:03Z","updated_at":"2015-06-15T15:05:03Z","body":"The fix in #4527 makes sense to me. However, perhaps a test based on the example should be added since this seems not to be covered by any tests presently"}}