Skip to content
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

[GeoMechanicsApplication] Add validation on completeness of input for GeoSettlement and GeoFlow workflows #11608

Open
rfaasse opened this issue Sep 26, 2023 · 1 comment

Comments

@rfaasse
Copy link
Contributor

rfaasse commented Sep 26, 2023

As a user, I want a clear error message when my input for running a GeoFlow or GeoSettlement workflow is incomplete, such that I quickly know how to fix the issue.

The bare minimum that we probably need is something along the following lines:

  • Boundary conditions for all displacement components.
  • Boundary conditions for the water pressure.
  • Gravity loading should be present.
  • "Appropriate" material parameters.

Acceptance criteria

Given the developer has processes defined in the ProjectParameters.json file
When the developer runs the settlement stage (using the KratosGeoSettlement::RunStage function)
Then if any of the minimum required processes associated to the settlement workflow is missing, raise an error "Missing necessary process (name of the process) for the GeoSettlement workflow, calculation is aborted".

Given the developer has processes defined in the ProjectParameters.json file
When the developer runs the geoflow stage (using the KratosExecute::ExecuteFlowAnalysis function)
Then if any of the minimum required processes associated to the settlement workflow is missing, raise an error "Missing necessary process (name of the process) for the GeoFlow workflow, calculation is aborted".

@rfaasse rfaasse converted this from a draft issue Sep 26, 2023
@rfaasse rfaasse changed the title Add validation on completeness of input for GeoSettlement and GeoFlow workflows [GeoMechanicsApplication] Add validation on completeness of input for GeoSettlement and GeoFlow workflows Sep 26, 2023
@rfaasse
Copy link
Contributor Author

rfaasse commented Sep 26, 2023

@indigocoral Moved the AC of the other issue (#11576) here. We might need a bit of discussion on the exact details of the acceptance criteria, but I think we can do that during refinement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant