{"action":"created","issue":{"url":"https://api.github.com/repos/thSoft/elysium/issues/98","labels_url":"https://api.github.com/repos/thSoft/elysium/issues/98/labels{/name}","comments_url":"https://api.github.com/repos/thSoft/elysium/issues/98/comments","events_url":"https://api.github.com/repos/thSoft/elysium/issues/98/events","html_url":"https://github.com/thSoft/elysium/pull/98","id":94285296,"number":98,"title":"exclude .ily from comlpete compilation and speed up build process","user":{"login":"nittka","id":2562651,"avatar_url":"https://avatars.githubusercontent.com/u/2562651?v=3","gravatar_id":"","url":"https://api.github.com/users/nittka","html_url":"https://github.com/nittka","followers_url":"https://api.github.com/users/nittka/followers","following_url":"https://api.github.com/users/nittka/following{/other_user}","gists_url":"https://api.github.com/users/nittka/gists{/gist_id}","starred_url":"https://api.github.com/users/nittka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nittka/subscriptions","organizations_url":"https://api.github.com/users/nittka/orgs","repos_url":"https://api.github.com/users/nittka/repos","events_url":"https://api.github.com/users/nittka/events{/privacy}","received_events_url":"https://api.github.com/users/nittka/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-10T11:00:56Z","updated_at":"2015-07-11T10:54:36Z","closed_at":"2015-07-11T10:53:45Z","pull_request":{"url":"https://api.github.com/repos/thSoft/elysium/pulls/98","html_url":"https://github.com/thSoft/elysium/pull/98","diff_url":"https://github.com/thSoft/elysium/pull/98.diff","patch_url":"https://github.com/thSoft/elysium/pull/98.patch"},"body":"This is a proposed fix for #95. The compile job for .ily files still has to be created in order to remove old markers but I removed all steps I thought to be unnecessary within the job. I sorted the files for compilation so that fast jobs (.ily) are handled first. This keeps the list of open jobs short (progress view).\r\n\r\nPreviously a new resource set was used for each direct include calculation. Changing that to one common resource set for each build speeds up the process. For the same reason I introduced the set of files for which this calculation has been done already (the filesToBuild-set does not quite serve the same purpose, it does not prevent duplicate calculation). As a result (on my slow Windows machine), the time for calculating allIncludingFiles dropped from 45 seconds to 4 seconds (for real-world/Mozart-KV250)."},"comment":{"url":"https://api.github.com/repos/thSoft/elysium/issues/comments/120603727","html_url":"https://github.com/thSoft/elysium/pull/98#issuecomment-120603727","issue_url":"https://api.github.com/repos/thSoft/elysium/issues/98","id":120603727,"user":{"login":"thSoft","id":118109,"avatar_url":"https://avatars.githubusercontent.com/u/118109?v=3","gravatar_id":"","url":"https://api.github.com/users/thSoft","html_url":"https://github.com/thSoft","followers_url":"https://api.github.com/users/thSoft/followers","following_url":"https://api.github.com/users/thSoft/following{/other_user}","gists_url":"https://api.github.com/users/thSoft/gists{/gist_id}","starred_url":"https://api.github.com/users/thSoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thSoft/subscriptions","organizations_url":"https://api.github.com/users/thSoft/orgs","repos_url":"https://api.github.com/users/thSoft/repos","events_url":"https://api.github.com/users/thSoft/events{/privacy}","received_events_url":"https://api.github.com/users/thSoft/received_events","type":"User","site_admin":false},"created_at":"2015-07-11T10:54:36Z","updated_at":"2015-07-11T10:54:36Z","body":"This makes more real-world integration tests pass and editor performance enhance! Thanks!"}}