{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/16571","repository_url":"https://api.github.com/repos/cms-sw/cmssw","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16571/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16571/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16571/events","html_url":"https://github.com/cms-sw/cmssw/pull/16571","id":189089795,"number":16571,"title":"Replace Subtraction Solid with Union Solid","user":{"login":"ianna","id":1390682,"avatar_url":"https://avatars.githubusercontent.com/u/1390682?v=3","gravatar_id":"","url":"https://api.github.com/users/ianna","html_url":"https://github.com/ianna","followers_url":"https://api.github.com/users/ianna/followers","following_url":"https://api.github.com/users/ianna/following{/other_user}","gists_url":"https://api.github.com/users/ianna/gists{/gist_id}","starred_url":"https://api.github.com/users/ianna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianna/subscriptions","organizations_url":"https://api.github.com/users/ianna/orgs","repos_url":"https://api.github.com/users/ianna/repos","events_url":"https://api.github.com/users/ianna/events{/privacy}","received_events_url":"https://api.github.com/users/ianna/received_events","type":"User","site_admin":false},"labels":[{"id":88653795,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-pending","name":"comparison-pending","color":"fbca04","default":false},{"id":53149121,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/fully-signed","name":"fully-signed","color":"009800","default":false},{"id":52882317,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/geometry-approved","name":"geometry-approved","color":"009800","default":false},{"id":56953091,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04","default":false},{"id":56131195,"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"009800","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_8_1_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/59/labels","id":1590252,"number":59,"title":"Next CMSSW_8_1_X","description":"","creator":{"login":"smuzaffar","id":4115138,"avatar_url":"https://avatars.githubusercontent.com/u/4115138?v=3","gravatar_id":"","url":"https://api.github.com/users/smuzaffar","html_url":"https://github.com/smuzaffar","followers_url":"https://api.github.com/users/smuzaffar/followers","following_url":"https://api.github.com/users/smuzaffar/following{/other_user}","gists_url":"https://api.github.com/users/smuzaffar/gists{/gist_id}","starred_url":"https://api.github.com/users/smuzaffar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smuzaffar/subscriptions","organizations_url":"https://api.github.com/users/smuzaffar/orgs","repos_url":"https://api.github.com/users/smuzaffar/repos","events_url":"https://api.github.com/users/smuzaffar/events{/privacy}","received_events_url":"https://api.github.com/users/smuzaffar/received_events","type":"User","site_admin":false},"open_issues":71,"closed_issues":1896,"state":"open","created_at":"2016-02-18T09:12:12Z","updated_at":"2016-11-14T11:30:52Z","due_on":"2016-11-01T00:00:00Z","closed_at":null},"comments":8,"created_at":"2016-11-14T11:30:29Z","updated_at":"2016-11-14T13:29:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/16571","html_url":"https://github.com/cms-sw/cmssw/pull/16571","diff_url":"https://github.com/cms-sw/cmssw/pull/16571.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/16571.patch"},"body":"* Replace Subtraction solid with Union solid whenever appropriate\r\n* Reduce subtracted Box size\r\n* Re-parent: place Pb volumes in Al layer\r\n\r\nBefore and after subtracted Boxes size:\r\n![screenshot 2016-11-11 15 23 23](https://cloud.githubusercontent.com/assets/1390682/20263154/af9a505e-aa65-11e6-92bb-9ba5d7ea308b.png)\r\n![screenshot 2016-11-11 16 20 27](https://cloud.githubusercontent.com/assets/1390682/20263155/af9e1bb2-aa65-11e6-8ae3-bd5df2cb6360.png)\r\n\r\nFinal shape does not change:\r\n![screenshot 2016-11-11 16 22 53](https://cloud.githubusercontent.com/assets/1390682/20263175/cba71fe8-aa65-11e6-9985-1b2f28ec383b.png)\r\n\r\nFinal description does not change, though more optimization with less Union solids could be done:\r\n![screenshot 2016-11-14 12 21 33](https://cloud.githubusercontent.com/assets/1390682/20263203/da1b8f28-aa65-11e6-8866-1f39ee9a43d5.png)\r\n"},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/260334395","html_url":"https://github.com/cms-sw/cmssw/pull/16571#issuecomment-260334395","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/16571","id":260334395,"user":{"login":"cmsbuild","id":4170616,"avatar_url":"https://avatars.githubusercontent.com/u/4170616?v=3","gravatar_id":"","url":"https://api.github.com/users/cmsbuild","html_url":"https://github.com/cmsbuild","followers_url":"https://api.github.com/users/cmsbuild/followers","following_url":"https://api.github.com/users/cmsbuild/following{/other_user}","gists_url":"https://api.github.com/users/cmsbuild/gists{/gist_id}","starred_url":"https://api.github.com/users/cmsbuild/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmsbuild/subscriptions","organizations_url":"https://api.github.com/users/cmsbuild/orgs","repos_url":"https://api.github.com/users/cmsbuild/repos","events_url":"https://api.github.com/users/cmsbuild/events{/privacy}","received_events_url":"https://api.github.com/users/cmsbuild/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T13:29:45Z","updated_at":"2016-11-14T13:29:45Z","body":"This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar"}}
{"actor":{"display_login":"cmsbuild"}}