{"action":"created","issue":{"url":"https://api.github.com/repos/modelica/fmi-standard/issues/100","repository_url":"https://api.github.com/repos/modelica/fmi-standard","labels_url":"https://api.github.com/repos/modelica/fmi-standard/issues/100/labels{/name}","comments_url":"https://api.github.com/repos/modelica/fmi-standard/issues/100/comments","events_url":"https://api.github.com/repos/modelica/fmi-standard/issues/100/events","html_url":"https://github.com/modelica/fmi-standard/issues/100","id":370765660,"node_id":"MDU6SXNzdWUzNzA3NjU2NjA=","number":100,"title":"Default value for variability in Section 2.2.7","user":{"login":"modelica-trac-importer","id":22202507,"node_id":"MDQ6VXNlcjIyMjAyNTA3","avatar_url":"https://avatars1.githubusercontent.com/u/22202507?v=4","gravatar_id":"","url":"https://api.github.com/users/modelica-trac-importer","html_url":"https://github.com/modelica-trac-importer","followers_url":"https://api.github.com/users/modelica-trac-importer/followers","following_url":"https://api.github.com/users/modelica-trac-importer/following{/other_user}","gists_url":"https://api.github.com/users/modelica-trac-importer/gists{/gist_id}","starred_url":"https://api.github.com/users/modelica-trac-importer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelica-trac-importer/subscriptions","organizations_url":"https://api.github.com/users/modelica-trac-importer/orgs","repos_url":"https://api.github.com/users/modelica-trac-importer/repos","events_url":"https://api.github.com/users/modelica-trac-importer/events{/privacy}","received_events_url":"https://api.github.com/users/modelica-trac-importer/received_events","type":"User","site_admin":false},"labels":[{"id":1093397692,"node_id":"MDU6TGFiZWwxMDkzMzk3Njky","url":"https://api.github.com/repos/modelica/fmi-standard/labels/P:%20minor","name":"P: minor","color":"dcffff","default":false},{"id":1046392993,"node_id":"MDU6TGFiZWwxMDQ2MzkyOTkz","url":"https://api.github.com/repos/modelica/fmi-standard/labels/enhancement","name":"enhancement","color":"a2eeef","default":true},{"id":1093395658,"node_id":"MDU6TGFiZWwxMDkzMzk1NjU4","url":"https://api.github.com/repos/modelica/fmi-standard/labels/specification","name":"specification","color":"0000aa","default":false}],"state":"open","locked":false,"assignee":{"login":"TorstenBlochwitz","id":33951596,"node_id":"MDQ6VXNlcjMzOTUxNTk2","avatar_url":"https://avatars1.githubusercontent.com/u/33951596?v=4","gravatar_id":"","url":"https://api.github.com/users/TorstenBlochwitz","html_url":"https://github.com/TorstenBlochwitz","followers_url":"https://api.github.com/users/TorstenBlochwitz/followers","following_url":"https://api.github.com/users/TorstenBlochwitz/following{/other_user}","gists_url":"https://api.github.com/users/TorstenBlochwitz/gists{/gist_id}","starred_url":"https://api.github.com/users/TorstenBlochwitz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TorstenBlochwitz/subscriptions","organizations_url":"https://api.github.com/users/TorstenBlochwitz/orgs","repos_url":"https://api.github.com/users/TorstenBlochwitz/repos","events_url":"https://api.github.com/users/TorstenBlochwitz/events{/privacy}","received_events_url":"https://api.github.com/users/TorstenBlochwitz/received_events","type":"User","site_admin":false},"assignees":[{"login":"TorstenBlochwitz","id":33951596,"node_id":"MDQ6VXNlcjMzOTUxNTk2","avatar_url":"https://avatars1.githubusercontent.com/u/33951596?v=4","gravatar_id":"","url":"https://api.github.com/users/TorstenBlochwitz","html_url":"https://github.com/TorstenBlochwitz","followers_url":"https://api.github.com/users/TorstenBlochwitz/followers","following_url":"https://api.github.com/users/TorstenBlochwitz/following{/other_user}","gists_url":"https://api.github.com/users/TorstenBlochwitz/gists{/gist_id}","starred_url":"https://api.github.com/users/TorstenBlochwitz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TorstenBlochwitz/subscriptions","organizations_url":"https://api.github.com/users/TorstenBlochwitz/orgs","repos_url":"https://api.github.com/users/TorstenBlochwitz/repos","events_url":"https://api.github.com/users/TorstenBlochwitz/events{/privacy}","received_events_url":"https://api.github.com/users/TorstenBlochwitz/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/modelica/fmi-standard/milestones/12","html_url":"https://github.com/modelica/fmi-standard/milestone/12","labels_url":"https://api.github.com/repos/modelica/fmi-standard/milestones/12/labels","id":3743800,"node_id":"MDk6TWlsZXN0b25lMzc0MzgwMA==","number":12,"title":"FMI 2.0","description":"","creator":{"login":"modelica-trac-importer","id":22202507,"node_id":"MDQ6VXNlcjIyMjAyNTA3","avatar_url":"https://avatars1.githubusercontent.com/u/22202507?v=4","gravatar_id":"","url":"https://api.github.com/users/modelica-trac-importer","html_url":"https://github.com/modelica-trac-importer","followers_url":"https://api.github.com/users/modelica-trac-importer/followers","following_url":"https://api.github.com/users/modelica-trac-importer/following{/other_user}","gists_url":"https://api.github.com/users/modelica-trac-importer/gists{/gist_id}","starred_url":"https://api.github.com/users/modelica-trac-importer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelica-trac-importer/subscriptions","organizations_url":"https://api.github.com/users/modelica-trac-importer/orgs","repos_url":"https://api.github.com/users/modelica-trac-importer/repos","events_url":"https://api.github.com/users/modelica-trac-importer/events{/privacy}","received_events_url":"https://api.github.com/users/modelica-trac-importer/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":47,"state":"closed","created_at":"2018-10-16T17:12:02Z","updated_at":"2018-10-16T19:32:56Z","due_on":"2012-12-17T08:00:00Z","closed_at":"2018-10-16T17:12:02Z"},"comments":3,"created_at":"2018-10-16T19:32:56Z","updated_at":"2018-10-16T19:33:37Z","closed_at":null,"author_association":"COLLABORATOR","body":"**Reported by iakovn on 25 Sep 2012 05:49 UTC**\nProposal: The default is “continuous” for Real variables and \"discrete\" in all other cases.\nMotivation: Current proposal is to always have default \"continuous\", but this value is only allowed for Real variables. Providing a valid default will reduce the size of XML for discrete models.\n\n\n----\n\nMigrated-From: https://trac.fmi-standard.org/ticket/100"},"comment":{"url":"https://api.github.com/repos/modelica/fmi-standard/issues/comments/430370076","html_url":"https://github.com/modelica/fmi-standard/issues/100#issuecomment-430370076","issue_url":"https://api.github.com/repos/modelica/fmi-standard/issues/100","id":430370076,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMDM3MDA3Ng==","user":{"login":"modelica-trac-importer","id":22202507,"node_id":"MDQ6VXNlcjIyMjAyNTA3","avatar_url":"https://avatars1.githubusercontent.com/u/22202507?v=4","gravatar_id":"","url":"https://api.github.com/users/modelica-trac-importer","html_url":"https://github.com/modelica-trac-importer","followers_url":"https://api.github.com/users/modelica-trac-importer/followers","following_url":"https://api.github.com/users/modelica-trac-importer/following{/other_user}","gists_url":"https://api.github.com/users/modelica-trac-importer/gists{/gist_id}","starred_url":"https://api.github.com/users/modelica-trac-importer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/modelica-trac-importer/subscriptions","organizations_url":"https://api.github.com/users/modelica-trac-importer/orgs","repos_url":"https://api.github.com/users/modelica-trac-importer/repos","events_url":"https://api.github.com/users/modelica-trac-importer/events{/privacy}","received_events_url":"https://api.github.com/users/modelica-trac-importer/received_events","type":"User","site_admin":false},"created_at":"2018-10-16T19:33:37Z","updated_at":"2018-10-16T19:33:37Z","author_association":"COLLABORATOR","body":"**Comment by iakovn on 15 Dec 2012 20:40 UTC**\nJust to clarify. In the XML of a large discrete model (>1e4 integer vars) there will be many lines of the kind\n```\n<ScalarVariable \n  name=\"i123456789101234567890\" \n  valueReference=\"1234567890\"\n  variability=\"discrete\">\n <Integer/>\n</ScalarVariable>\n```\nThe sub-string 'variability=\"discrete\"' will have to be repeated and may take up to 20% of the file size. Introduction of the proper default will improve XML parsing performance without any side effects.\n\nThe ticket is therefore a minor performance improvement. I cannot see how this is a major change. I believe, it is a common practice to have valid defaults where possible in order to optimize parsing performance.\n\nReplying to [comment:3 andreas.nicolai@…]:\n> An explicit specification of continuous/discrete behavior really only makes sense for Real variables. So the question arises why we need a default for integer/bool/string values at all? \n\nI don't really understand the comment. Variability has 5 different values (not just continuous/discrete). Please, clarify.\n"}}
{"actor":{"display_login":"modelica-trac-importer"}}