{"action":"created","issue":{"url":"https://api.github.com/repos/idaholab/moose/issues/5255","labels_url":"https://api.github.com/repos/idaholab/moose/issues/5255/labels{/name}","comments_url":"https://api.github.com/repos/idaholab/moose/issues/5255/comments","events_url":"https://api.github.com/repos/idaholab/moose/issues/5255/events","html_url":"https://github.com/idaholab/moose/pull/5255","id":88895819,"number":5255,"title":"Change getMaterialProperty semantics to match getFunction/getPostprocessorValue","user":{"login":"dschwen","id":202302,"avatar_url":"https://avatars.githubusercontent.com/u/202302?v=3","gravatar_id":"","url":"https://api.github.com/users/dschwen","html_url":"https://github.com/dschwen","followers_url":"https://api.github.com/users/dschwen/followers","following_url":"https://api.github.com/users/dschwen/following{/other_user}","gists_url":"https://api.github.com/users/dschwen/gists{/gist_id}","starred_url":"https://api.github.com/users/dschwen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dschwen/subscriptions","organizations_url":"https://api.github.com/users/dschwen/orgs","repos_url":"https://api.github.com/users/dschwen/repos","events_url":"https://api.github.com/users/dschwen/events{/privacy}","received_events_url":"https://api.github.com/users/dschwen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":15,"created_at":"2015-06-17T03:00:29Z","updated_at":"2015-06-17T18:12:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/idaholab/moose/pulls/5255","html_url":"https://github.com/idaholab/moose/pull/5255","diff_url":"https://github.com/idaholab/moose/pull/5255.diff","patch_url":"https://github.com/idaholab/moose/pull/5255.patch"},"body":"This is a fairly large commit that closes #5218. It augments ```getMaterialProperty``` to allow a user to pass in an input parameter key for an input parameter of type ```MaterialPropertyName```. This avoids the common ```getMaterialProperty<>(getParam<std::string>(\"key\"))``` pattern and replaces it with ```getMaterialProperty<>(\"key\")```. The big benefit is that the input parameters of type ```MaterialPropertyName``` now accept Real numbers for Real material properties. This allows developers to set numerical default values for material properties which are taken if no material property name is provided. It also allows users to supply constant numbers in input files directly where material properties of type real are needed.\r\n\r\nThere might be some additional application patches necessary (a one line patch is already in Marmot, Grizzly didn't need anything, I'm working on hyrax). I'll work with @snschune tomorrow to get yak and rattlesnake in shape.\r\n\r\nI have a few commits in place to use the new semantics all over framework, modules and tests (btw. the default property stuff is covered in the tests now). I have also edited some input files to remove a few GenericConstantMaterial properties and use default material properties instead. I'm sure more can be done (especially on the applications side).\r\n\r\nI will add follow on PRs (for new issues) to update the derivative material property system accordingly, but this PR is big enough as it is."},"comment":{"url":"https://api.github.com/repos/idaholab/moose/issues/comments/112900322","html_url":"https://github.com/idaholab/moose/pull/5255#issuecomment-112900322","issue_url":"https://api.github.com/repos/idaholab/moose/issues/5255","id":112900322,"user":{"login":"amjokisaari","id":6676372,"avatar_url":"https://avatars.githubusercontent.com/u/6676372?v=3","gravatar_id":"","url":"https://api.github.com/users/amjokisaari","html_url":"https://github.com/amjokisaari","followers_url":"https://api.github.com/users/amjokisaari/followers","following_url":"https://api.github.com/users/amjokisaari/following{/other_user}","gists_url":"https://api.github.com/users/amjokisaari/gists{/gist_id}","starred_url":"https://api.github.com/users/amjokisaari/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amjokisaari/subscriptions","organizations_url":"https://api.github.com/users/amjokisaari/orgs","repos_url":"https://api.github.com/users/amjokisaari/repos","events_url":"https://api.github.com/users/amjokisaari/events{/privacy}","received_events_url":"https://api.github.com/users/amjokisaari/received_events","type":"User","site_admin":false},"created_at":"2015-06-17T18:12:30Z","updated_at":"2015-06-17T18:12:30Z","body":"great, thanks.\n\n\n\nOn 15-06-17 01:02 PM, Daniel Schwen wrote:\n> Just merge my PR once this moose PR is in\n>\n> On Wed, Jun 17, 2015, 9:43 AM A. M. Jokisaari <notifications@github.com>\n> wrote:\n>\n> > @dschwen <https://github.com/dschwen>, this is nifty, what needs to be\n> > done on my end for hyrax?\n> >\n> > —\n> > Reply to this email directly or view it on GitHub\n> > <https://github.com/idaholab/moose/pull/5255#issuecomment-112855092>.\n> >\n>\n> —\n> Reply to this email directly or view it on GitHub \n> <https://github.com/idaholab/moose/pull/5255#issuecomment-112877867>.\n>\n\n"}}