{"action":"created","issue":{"url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/472","labels_url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/472/labels{/name}","comments_url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/472/comments","events_url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/472/events","html_url":"https://github.com/lbl-srg/modelica-buildings/issues/472","id":120964211,"number":472,"title":"Changed extractor implementation for increased robustness in pumps","user":{"login":"Mathadon","id":5970879,"avatar_url":"https://avatars.githubusercontent.com/u/5970879?v=3","gravatar_id":"","url":"https://api.github.com/users/Mathadon","html_url":"https://github.com/Mathadon","followers_url":"https://api.github.com/users/Mathadon/followers","following_url":"https://api.github.com/users/Mathadon/following{/other_user}","gists_url":"https://api.github.com/users/Mathadon/gists{/gist_id}","starred_url":"https://api.github.com/users/Mathadon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mathadon/subscriptions","organizations_url":"https://api.github.com/users/Mathadon/orgs","repos_url":"https://api.github.com/users/Mathadon/repos","events_url":"https://api.github.com/users/Mathadon/events{/privacy}","received_events_url":"https://api.github.com/users/Mathadon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-08T09:04:27Z","updated_at":"2015-12-09T02:45:20Z","closed_at":null,"body":"I recently came across an issue where that the `extractor` block implementation seems to be malfunctioning:\r\n\r\n![screen shot 2015-12-07 at 10 35 24](https://cloud.githubusercontent.com/assets/5970879/11651647/6caf1a00-9db8-11e5-8a14-6fbe4f8d62cc.png)\r\n\r\nAs you can see the `k[2]` value changes even though the `index` input does not change. Also the initialisation is wrong. It's difficult to get a simple example to reproduce this so in the meantime I'd like to propose a new implementation of `extractor` that does not rely on the `when change(index)` construct. Hopefully I can afterwards make a bug report a dymola or MSL.\r\n\r\nI'll make a pull request."},"comment":{"url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/comments/163088879","html_url":"https://github.com/lbl-srg/modelica-buildings/issues/472#issuecomment-163088879","issue_url":"https://api.github.com/repos/lbl-srg/modelica-buildings/issues/472","id":163088879,"user":{"login":"mwetter","id":1329080,"avatar_url":"https://avatars.githubusercontent.com/u/1329080?v=3","gravatar_id":"","url":"https://api.github.com/users/mwetter","html_url":"https://github.com/mwetter","followers_url":"https://api.github.com/users/mwetter/followers","following_url":"https://api.github.com/users/mwetter/following{/other_user}","gists_url":"https://api.github.com/users/mwetter/gists{/gist_id}","starred_url":"https://api.github.com/users/mwetter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwetter/subscriptions","organizations_url":"https://api.github.com/users/mwetter/orgs","repos_url":"https://api.github.com/users/mwetter/repos","events_url":"https://api.github.com/users/mwetter/events{/privacy}","received_events_url":"https://api.github.com/users/mwetter/received_events","type":"User","site_admin":false},"created_at":"2015-12-09T02:45:20Z","updated_at":"2015-12-09T02:45:20Z","body":"@Mathadon It probably is best to implement your work-around as a protected block inside the model that uses the extractor. This way, no other model can use it and hence we can remove it from the library when it is fixed in dymola without causing backward incompatibility."}}