{"action":"created","issue":{"url":"https://api.github.com/repos/prusa3d/Slic3r/issues/32","repository_url":"https://api.github.com/repos/prusa3d/Slic3r","labels_url":"https://api.github.com/repos/prusa3d/Slic3r/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/prusa3d/Slic3r/issues/32/comments","events_url":"https://api.github.com/repos/prusa3d/Slic3r/issues/32/events","html_url":"https://github.com/prusa3d/Slic3r/issues/32","id":188921961,"number":32,"title":"Thin wall: Gapfill not working","user":{"login":"Sebastianv650","id":10776877,"avatar_url":"https://avatars.githubusercontent.com/u/10776877?v=3","gravatar_id":"","url":"https://api.github.com/users/Sebastianv650","html_url":"https://github.com/Sebastianv650","followers_url":"https://api.github.com/users/Sebastianv650/followers","following_url":"https://api.github.com/users/Sebastianv650/following{/other_user}","gists_url":"https://api.github.com/users/Sebastianv650/gists{/gist_id}","starred_url":"https://api.github.com/users/Sebastianv650/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sebastianv650/subscriptions","organizations_url":"https://api.github.com/users/Sebastianv650/orgs","repos_url":"https://api.github.com/users/Sebastianv650/repos","events_url":"https://api.github.com/users/Sebastianv650/events{/privacy}","received_events_url":"https://api.github.com/users/Sebastianv650/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-11-12T16:59:39Z","updated_at":"2016-11-12T18:10:47Z","closed_at":"2016-11-12T18:05:27Z","body":"### Version\r\nc1af9bda0a9fcf2703c5847fbd9d43c62c7f9c6c\r\n\r\n### Operating system type + version\r\nWin 10 64bit\r\n\r\n### Behavior\r\nEven with detect thin walls enabled, Slic3r tries to fill the thin walls of the model in the upper area with 2 perimeters, that would result in masive over-extrusion. A few layers down, where the wall thickness is a little bit lower, it creates 1 perimeter with a gapfill-infill as desired.\r\n3D Preview of the over-filled layer, you can baraly see the 2 perimeters:\r\n![prusa](https://cloud.githubusercontent.com/assets/10776877/20239476/1c5e7b72-a901-11e6-903a-82822682ec9e.JPG)\r\n2D View of the layer:\r\n![prusa_2](https://cloud.githubusercontent.com/assets/10776877/20239479/26375024-a901-11e6-851e-e69fb2e0543e.JPG)\r\n\r\nAlexrj's Slic3r did a good job on the model, same layer, same settings:\r\n![orig](https://cloud.githubusercontent.com/assets/10776877/20239485/34f474f2-a901-11e6-9d8b-325fb9fc30cc.JPG)\r\n\r\nIt looks to me as in prusa3d-Slic3r the gap fill line width is limited to a max. value. If that line width is exceeded, it tries to use normal perimeters without a check that this can't work?\r\nExcelent gapfill of thin walls is the main reason I love Slic3r, so it would be great if this version can keep this feature at the same quality :-)\r\n\r\nAnother thing I noticed during slicing the model: alexrj Slic3r says Manifold: yes. Prusa Slic3r is stating over 800 errors in the model? Might be related, but maybe not..\r\n\r\n#### STL/Config (.ZIP) where problem occurs\r\nhttp://www.thingiverse.com/thing:1817180\r\n"},"comment":{"url":"https://api.github.com/repos/prusa3d/Slic3r/issues/comments/260138564","html_url":"https://github.com/prusa3d/Slic3r/issues/32#issuecomment-260138564","issue_url":"https://api.github.com/repos/prusa3d/Slic3r/issues/32","id":260138564,"user":{"login":"bubnikv","id":5830947,"avatar_url":"https://avatars.githubusercontent.com/u/5830947?v=3","gravatar_id":"","url":"https://api.github.com/users/bubnikv","html_url":"https://github.com/bubnikv","followers_url":"https://api.github.com/users/bubnikv/followers","following_url":"https://api.github.com/users/bubnikv/following{/other_user}","gists_url":"https://api.github.com/users/bubnikv/gists{/gist_id}","starred_url":"https://api.github.com/users/bubnikv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bubnikv/subscriptions","organizations_url":"https://api.github.com/users/bubnikv/orgs","repos_url":"https://api.github.com/users/bubnikv/repos","events_url":"https://api.github.com/users/bubnikv/events{/privacy}","received_events_url":"https://api.github.com/users/bubnikv/received_events","type":"User","site_admin":false},"created_at":"2016-11-12T18:10:47Z","updated_at":"2016-11-12T18:10:47Z","body":"Thanks, I have reverted a recent commit, which causes this behavior. I did not quite understand the code, now I believe I do much better. Learning by a trial and error, but I am slowly getting confident with the Slic3r internals. It took quite a long time indeed. It is also an answer, why there are not many contributors to Slic3r, as it takes quite some time to work in.\r\n\r\nWe at Prusa3D have the \"detect thin walls\" off as it introduces unnecessary roundings of the perimeters, which then need to be filled in by the not very reliable gap fill. There is still room for improvement indeed.\r\n\r\nThanks for keeping watch over this fork and for doing a great testing job. I hope with this commit we can release a useful build to the world.\r\n"}}
{"actor":{"display_login":"bubnikv"}}