{"action":"created","issue":{"url":"https://api.github.com/repos/tbeu/Modelica/issues/795","repository_url":"https://api.github.com/repos/tbeu/Modelica","labels_url":"https://api.github.com/repos/tbeu/Modelica/issues/795/labels{/name}","comments_url":"https://api.github.com/repos/tbeu/Modelica/issues/795/comments","events_url":"https://api.github.com/repos/tbeu/Modelica/issues/795/events","html_url":"https://github.com/tbeu/Modelica/issues/795","id":197529174,"number":795,"title":"Physical types in Modelica.SIunits should not have start, min, or max values.","user":{"login":"modelica-trac-importer","id":22202507,"avatar_url":"https://avatars.githubusercontent.com/u/22202507?v=3","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":506762997,"url":"https://api.github.com/repos/tbeu/Modelica/labels/bug","name":"bug","color":"ee0701","default":true},{"id":506769806,"url":"https://api.github.com/repos/tbeu/Modelica/labels/L:SIunits","name":"L:SIunits","color":"0000aa","default":false},{"id":506770144,"url":"https://api.github.com/repos/tbeu/Modelica/labels/P:%20high","name":"P: high","color":"ff6666","default":false}],"state":"closed","locked":false,"assignee":{"login":"beutlich","id":14896695,"avatar_url":"https://avatars.githubusercontent.com/u/14896695?v=3","gravatar_id":"","url":"https://api.github.com/users/beutlich","html_url":"https://github.com/beutlich","followers_url":"https://api.github.com/users/beutlich/followers","following_url":"https://api.github.com/users/beutlich/following{/other_user}","gists_url":"https://api.github.com/users/beutlich/gists{/gist_id}","starred_url":"https://api.github.com/users/beutlich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beutlich/subscriptions","organizations_url":"https://api.github.com/users/beutlich/orgs","repos_url":"https://api.github.com/users/beutlich/repos","events_url":"https://api.github.com/users/beutlich/events{/privacy}","received_events_url":"https://api.github.com/users/beutlich/received_events","type":"User","site_admin":false},"assignees":[{"login":"beutlich","id":14896695,"avatar_url":"https://avatars.githubusercontent.com/u/14896695?v=3","gravatar_id":"","url":"https://api.github.com/users/beutlich","html_url":"https://github.com/beutlich","followers_url":"https://api.github.com/users/beutlich/followers","following_url":"https://api.github.com/users/beutlich/following{/other_user}","gists_url":"https://api.github.com/users/beutlich/gists{/gist_id}","starred_url":"https://api.github.com/users/beutlich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beutlich/subscriptions","organizations_url":"https://api.github.com/users/beutlich/orgs","repos_url":"https://api.github.com/users/beutlich/repos","events_url":"https://api.github.com/users/beutlich/events{/privacy}","received_events_url":"https://api.github.com/users/beutlich/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/tbeu/Modelica/milestones/47","html_url":"https://github.com/tbeu/Modelica/milestone/47","labels_url":"https://api.github.com/repos/tbeu/Modelica/milestones/47/labels","id":2214850,"number":47,"title":"MSL3.2.1","description":"Backward compatible bug-fix release.","creator":{"login":"modelica-trac-importer","id":22202507,"avatar_url":"https://avatars.githubusercontent.com/u/22202507?v=3","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":0,"closed_issues":119,"state":"closed","created_at":"2016-12-25T13:00:27Z","updated_at":"2016-12-26T03:26:42Z","due_on":"2013-03-31T07:00:00Z","closed_at":"2016-12-25T13:00:27Z"},"comments":19,"created_at":"2016-12-26T03:21:58Z","updated_at":"2016-12-26T03:26:52Z","closed_at":"2016-12-26T03:26:42Z","body":"**Reported by stefanv on 23 Aug 2012 15:16 UTC**\nThere are several types in Modelica.SIunits to which start, min, and max values have been added some time in the 3.x timeframe. These do not belong in the basic definitions of such physical quantities (exceptions listed below), as they are invariably wrong for any given application. Such modifications belong in the definitions of library or user models that use these basic types.\n\nThe offending types are Pressure, AbsolutePressure, PressureDifference, BulkModulus, ThermodynamicTemperature, TemperatureDifference, MassFraction, and MoleFraction.\n\nExceptions:\n\nmin=0 is probably reasonable for ThermodynamicTemperature.\n\nmin=0,max=1 is probably reasonable for MassFraction and MoleFraction.\n\n----\n\nMigrated-From: https://trac.modelica.org/Modelica/ticket/795"},"comment":{"url":"https://api.github.com/repos/tbeu/Modelica/issues/comments/269154531","html_url":"https://github.com/tbeu/Modelica/issues/795#issuecomment-269154531","issue_url":"https://api.github.com/repos/tbeu/Modelica/issues/795","id":269154531,"user":{"login":"modelica-trac-importer","id":22202507,"avatar_url":"https://avatars.githubusercontent.com/u/22202507?v=3","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":"2016-12-26T03:26:52Z","updated_at":"2016-12-26T03:26:52Z","body":"**Comment by otter on 1 Jan 2013 10:54 UTC**\nMade one change in 60207641e1445efb836dfb7effd733065bc5461e:\n\nRemoved \"nominal\" from \"Pressure\" and included \"nominal=1e5\" in AbsolutePressure.\n\nReason: \"Pressure\" is used both for relative and absolute pressure.\nFor relative pressure a nominal value of 1e5 is quite arbitray and might\nbe often not a good value. For an absolute pressure, a nominal value of\nthe standard atmospheric pressure on earth is in nearly all applications\nbetter as the default of nominal = 1."}}
{"actor":{"display_login":"modelica-trac-importer"}}