Ben Spencer

Results 29 comments of Ben Spencer

Actually, seems like the documentation failure might have been unrelated to these changes. I guess the main missing thing is tests.

This is really useful. I would suggest that an option should be added to specify which block the sideset should be with respect to. Sometimes these sidesets come in with...

> That's a good suggestion, can we add this in a separate issue because this one has taken long time? That's fine!

This looks really helpful! Sorry to continue with my tradition of nit-picking naming choices, but my reaction when I saw this is that `Capabilities` is a very broad term, and...

My update was simply to run clang-format because the version Austin has on his machine behaves a little differently.

I agree that we need to straighten things out, and it sounds like we should be able to see some big performance gains when we get this right for contact...

I've long thought that we should rename `heat_conduction` to `heat_transfer` I appreciate @lcarasik's input as someone who is more knowledgable in that field, though, so if `heat_transfer` is misleading, I...

On further inspection of this code, it is pretty clear that the models that we have now in BlackBear are a proof of concept. Since the time they were developed,...

Let's use this issue for general test maintenance such as removing deprecated syntax too.

This is the Grizzly PR that will rely on this: https://github.inl.gov/ncrc/grizzly/pull/765