Skip to content

Commit

Permalink
Remove useless comments/function
Browse files Browse the repository at this point in the history
  • Loading branch information
blaisb committed Dec 4, 2024
1 parent fd49ba9 commit e3d8335
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 19 deletions.
21 changes: 6 additions & 15 deletions include/fem-dem/void_fraction.h
Original file line number Diff line number Diff line change
Expand Up @@ -148,20 +148,10 @@ class VoidFractionBase : public PhysicsLinearSubequationsSolver


/**
* @brief Assemble the diagonal of the mass matrix to impose constraints on the system
* @brief Calculates the void fraction
*
* @param[out] diagonal_mass_matrix The matrix for which the diagonal entries will be filled with the diagonal of the mass matrix
*
* @todo Establish if it is really necessary to keep this as a matrix and not as a vector since a diagonal is nothing more than a vector.
*/
void
assemble_mass_matrix_diagonal(
TrilinosWrappers::SparseMatrix &diagonal_mass_matrix);

/**
* @brief Calculate the void fraction.
*
* @param[in] time Current time for which the void fraction is to be calculated.
* @param[in] time Current time for which the void fraction is to be
* calculated.
*
*/
void
Expand Down Expand Up @@ -237,7 +227,8 @@ class VoidFractionBase : public PhysicsLinearSubequationsSolver
/**
* @brief Calculate the void fraction using a function. This is a straightforward usage of VectorTools.
*
* @param[in] time Current time for which the void fraction is to be calculated.
* @param[in] time Current time for which the void fraction is to be
* calculated.
*
*/
void
Expand Down Expand Up @@ -278,7 +269,7 @@ class VoidFractionBase : public PhysicsLinearSubequationsSolver
* @brief Calculate and return the periodic offset distance vector of the domain which is needed
* for the periodic boundary conditions using the QCM or SPM for void fraction
* with the GLS VANS/CFD-DEM solver. The distance is based on one of the
* periodic boundaries. This periodic boundary is then used as the reference
* periodic boundaries. This periodic boundary is then used as the reference
* for all particle locations shifted by this offset vector.
*
* @param[in] boundary_id The ID of one of the periodic boundaries
Expand Down
4 changes: 0 additions & 4 deletions source/fem-dem/void_fraction.cc
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ VoidFractionBase<dim>::setup_dofs()
DoFTools::make_hanging_node_constraints(dof_handler,
void_fraction_constraints);

/// TODO ADD Periodic constraints
void_fraction_constraints.close();

void_fraction_locally_relevant.reinit(locally_owned_dofs,
Expand Down Expand Up @@ -169,13 +168,10 @@ VoidFractionBase<dim>::calculate_void_fraction(const double time)
{
calculate_void_fraction_particle_centered_method();
}
// particle_centered_method();
else if (void_fraction_parameters->mode == Parameters::VoidFractionMode::qcm)
{
calculate_void_fraction_quadrature_centered_method();
}

// quadrature_centered_sphere_method(load_balance_step);
else if (void_fraction_parameters->mode == Parameters::VoidFractionMode::spm)
{
calculate_void_fraction_satellite_point_method();
Expand Down

0 comments on commit e3d8335

Please sign in to comment.