Skip to content

Conversation

@masterleinad
Copy link
Collaborator

@masterleinad masterleinad commented Jul 22, 2024

Copying the parallel layout from displacement, we are missing some locally relevant DoFs we are accessing in compute_stress.

@masterleinad masterleinad marked this pull request as ready for review July 22, 2024 21:30
@Rombur
Copy link
Member

Rombur commented Jul 23, 2024

An error occurred in line <1479> of file </scratch/source/dealii/include/deal.II/lac/la_parallel_vector.templates.h> in function
    dealii::LinearAlgebra::distributed::Vector<Number, MemorySpace>& dealii::LinearAlgebra::distributed::Vector<Number, MemorySpace>::operator-=(const dealii::LinearAlgebra::VectorSpaceVector<Number>&) [with Number = double; MemorySpace = dealii::MemorySpace::Host]
The violated condition was: 
    ::dealii::deal_II_exceptions::internals::compare_for_equality(locally_owned_size(), v.locally_owned_size())
Additional information: 
    Two sizes or dimensions were supposed to be equal, but aren't. They
    are 336 and 0.

Stacktrace:
-----------
#0  /opt/dealii/lib/libdeal_II.g.so.9.5.1: dealii::LinearAlgebra::distributed::Vector<double, dealii::MemorySpace::Host>::operator-=(dealii::LinearAlgebra::VectorSpaceVector<double> const&)
#1  /var/jenkins/workspace/adamantine_PR-303/build/bin/test_mechanical_physics: adamantine::MechanicalPhysics<3, 4, adamantine::SolidLiquidPowder, dealii::MemorySpace::Host>::solve()
#2  /var/jenkins/workspace/adamantine_PR-303/build/bin/test_mechanical_physics: elastostatic::test_method()

@masterleinad masterleinad force-pushed the compute_stress_relevant branch from a644f5a to 686422d Compare July 23, 2024 14:13
@masterleinad masterleinad marked this pull request as draft July 24, 2024 22:26
@masterleinad masterleinad force-pushed the compute_stress_relevant branch 3 times, most recently from 4baab5e to f2d286c Compare July 25, 2024 18:13
@masterleinad
Copy link
Collaborator Author

Retest this please.

@masterleinad masterleinad marked this pull request as ready for review July 29, 2024 21:15
_dof_handler.locally_owned_dofs(), locally_relevant_dofs,
_mechanical_operator->rhs().get_mpi_communicator());
incremental_displacement = displacement;
// TODO Remove this once we support refinement/material addition
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove the TODO. This PR fixes it.

@masterleinad masterleinad force-pushed the compute_stress_relevant branch from a4311fe to 96fd762 Compare July 30, 2024 13:04
@masterleinad
Copy link
Collaborator Author

Retest this please.

@masterleinad masterleinad force-pushed the compute_stress_relevant branch from 96fd762 to 773c53d Compare October 2, 2024 14:51
@Rombur Rombur merged commit a38a87b into adamantine-sim:master Oct 3, 2024
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

Successfully merging this pull request may close these issues.

2 participants