-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make "Scale BC Dofs: true" option with scaling computed on the fly default for LCM? #9
Comments
Yes, let's make this the default. |
Ok. Let me see what happens first if I make it the default, namely how many tests will need to be rebaselined. I'll post the results here. |
I started working on this. When I force the LCM problems to use 'Scale BC Dofs: true' with the automatically-generated scaling, I am getting the following test failures: Small tests:
Large tests:
Some of these seem to "fail" trivially (there are minor differences found by exodiff), but some give very different results. Specifically, it seems most (all?) of the tests that fail non-trivially give a fundamentally different continuation history, e.g., for the MechanicsWithHelium problem:
Does anyone have any insight into why that would be the case with the scaling? The nonlinear solves converge just fine with the scaling, so it is not like they are failing, leading to the strange behavior. The way I am doing this, one can always over-write the default turning on of the scaling from parameter list to run w/o scaling; but I would like to understand why this behavior happens for some of the tests with the scaling. @jwfoulk , @lxmota : any thoughts? |
This doesn't surprise me. It may be that the solutions are still correct. These problems are very sensitive to anything you change. We need to verify manually if they are correct and re-bless then. Not ideal, but they are too sensitive. |
This is a follow on issue to sandialabs/Albany#307 . Now that "Scale BC Dofs: true" is working correctly and I have added a version of it that computes / adapts the scaling on the fly, should we make this option default for LCM? One thing to consider is this sort of change may require rebaselining a lot of the tests that use exodiff (e.g., if the gold file has a residual field, which will change with the scaling).
The text was updated successfully, but these errors were encountered: