Skip to content

[Core][Analysis stage] Attempting to solve #13350 #13432

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RiccardoRossi
Copy link
Member

@RiccardoRossi RiccardoRossi commented May 15, 2025

📝 Description

This is the very first step towards attempting to solve #13350

@loumalouomega @AlejandroCornejo @mcgicjn2

This is definitely not ready for merging, it is just to open the iterations

Fixes #13350 (adding this to close issue automatically if this is merged)

🆕 Changelog

This is the very first step towards attempting to solve #13350

@loumalouomega @AlejandroCornejo @mcgicjn2
@RiccardoRossi RiccardoRossi requested a review from a team as a code owner May 15, 2025 10:16
@RiccardoRossi RiccardoRossi added Bug API Breaker Bugfix Behaviour Change changes behaviour but not the API labels May 15, 2025
@loumalouomega loumalouomega changed the title Attempting to solve #13350 [Core][Analysis stage] Attempting to solve #13350 May 15, 2025
@loumalouomega
Copy link
Member

I will wait for the @KratosMultiphysics/cosimulation and @KratosMultiphysics/geomechanics feedbacks, as well as CI.

@RiccardoRossi
Copy link
Member Author

luckily grepping for "GetSolver().Predict" gives relatively few calls in the repository:

kratos/python_scripts/analysis_stage.py:            self._GetSolver().Predict()
applications/GeoMechanicsApplication/python_scripts/piping_analysis.py:            self._GetSolver().Predict()
applications/GeoMechanicsApplication/python_scripts/geomechanics_analysis.py:                self._GetSolver().Predict()
applications/CoSimulationApplication/tests/structural_mechanics_analysis_remote_controlled.py:            self._GetSolver().Predict()    
applications/CoSimulationApplication/python_scripts/analysis_stage_with_co_sim_io.py:                self._GetSolver().Predict()           
applications/ConstitutiveLawsApplication/python_scripts/high_cycle_fatigue_analysis.py:            self._GetSolver().Predict()   
applications/SwimmingDEMApplication/python_scripts/DEM_coupled_pfem_fluid_dynamics_analysis.py:        self._GetSolver().Predict()
applications/OptimizationApplication/python_scripts/filtering/helmholtz_analysis.py:        self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()  
python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/adaptative_remeshing_structural_mechanics_analysis.py:                        self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/rve_analysis.py:        self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/structural_mechanics_prebuckling_analysis.py:            self._GetSolver().Predict()
applications/ContactStructuralMechanicsApplication/python_scripts/adaptative_remeshing_contact_structural_mechanics_analysis.py:                self._GetSolver().Predict()
applications/ContactStructuralMechanicsApplication/python_scripts/adaptative_remeshing_contact_structural_mechanics_analysis.py:                        self._GetSolver().Predict()  

so with some luck these are the only analysis stages we should look into

@rubenzorrilla
Copy link
Member

luckily grepping for "GetSolver().Predict" gives relatively few calls in the repository:

kratos/python_scripts/analysis_stage.py:            self._GetSolver().Predict()
applications/GeoMechanicsApplication/python_scripts/piping_analysis.py:            self._GetSolver().Predict()
applications/GeoMechanicsApplication/python_scripts/geomechanics_analysis.py:                self._GetSolver().Predict()
applications/CoSimulationApplication/tests/structural_mechanics_analysis_remote_controlled.py:            self._GetSolver().Predict()    
applications/CoSimulationApplication/python_scripts/analysis_stage_with_co_sim_io.py:                self._GetSolver().Predict()           
applications/ConstitutiveLawsApplication/python_scripts/high_cycle_fatigue_analysis.py:            self._GetSolver().Predict()   
applications/SwimmingDEMApplication/python_scripts/DEM_coupled_pfem_fluid_dynamics_analysis.py:        self._GetSolver().Predict()
applications/OptimizationApplication/python_scripts/filtering/helmholtz_analysis.py:        self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()  
python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxial_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_triaxials_and_bts_tests.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/DEMApplication/python_scripts/custom_material_tests/decompressed_material_bts_test.py:            self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/adaptative_remeshing_structural_mechanics_analysis.py:                        self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/rve_analysis.py:        self._GetSolver().Predict()
applications/StructuralMechanicsApplication/python_scripts/structural_mechanics_prebuckling_analysis.py:            self._GetSolver().Predict()
applications/ContactStructuralMechanicsApplication/python_scripts/adaptative_remeshing_contact_structural_mechanics_analysis.py:                self._GetSolver().Predict()
applications/ContactStructuralMechanicsApplication/python_scripts/adaptative_remeshing_contact_structural_mechanics_analysis.py:                        self._GetSolver().Predict()  

so with some luck these are the only analysis stages we should look into

These are great news. However, note that there could be some "non-standard" calls of the form of self.my_solver.Predict(). Though we'll need to check these one by one, I believe that they're very few.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Breaker Behaviour Change changes behaviour but not the API Bug Bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Major issue in the ordering of Predict and InitializeSolutionStep (and of ConvergenceCriterion initialization)
4 participants