Guillaume Giudicelli

Results 719 comments of Guillaume Giudicelli

Yes it is for the current transfers. The tests for the general field transfers will cover this case

Hello Now you will need to amend the commits to add "refs #issue_number" in them I think the PR should be only about QuadraturePointMultiApp. You can open another PR for...

@abarun22 please just make one test and I ll do the extension to all the other transfers. You'll need to fix the other PR to only include reset-related items. Also...

Hello This is what I mean https://git-scm.com/book/en/v2/Git-Branching-Rebasing

Ok good for the separate PR. I ll go help there when I have time. For this PR, the current test does not work. - the multiapp does not do...

Should be ready for another round or merge @roystgnr

@roystgnr should be good for re-approval please

Thanks for the prompt reviews!

@cpgr are you waiting for a review on this?