Skip to content
Snippets Groups Projects

Add IProductWRTBase operator and standard matrix implementation

Merged Add IProductWRTBase operator and standard matrix implementation
feature/IProdWRTBasePhysDerv into master
All threads resolved!
Merged João Isler requested to merge feature/IProdWRTBasePhysDerv into master
All threads resolved!

In this merge request, the IProductWRTBase operator is developed and the standard matrix implementation is also added.

The IProductWRTBase.hpp, IProductWRTBase.cpp, IProductWRTBaseImpl.cpp and IProductWRTBaseStdMat.hpp files were added for this operator. This implementation was based on the BwdTrans operator code. In the IProductWRTBaseStdMat.hpp, note that the jacobian is initialised in the constructor of the OperatorIProductWRTBaseImpl class for deformed and undeformed elements. Also, in the main.cpp file, input and output Field objects are created for the IProdWRTBase, and the code applies the operator for a mesh.

The InitFields class (init_fields.hpp) was generalised to be able to deal with all types of operators and mesh files. Thus, the InitFields class was templated with two parameters. The parameters are FieldStates to create the input and output Field objects for the operators. The mesh name is obtained through a virtual function which returns the chosen mesh name. In addition, a boost test case (test_ipwrtbase.cpp) was added to test the IProductWRTBase operator using the standard matrix implementation for a one-dimensional problem.

Two new meshes were also added to the project. The first one is a one-dimensional mesh which is used for the boost test case. The second is a two-dimensional mesh with deformed elements. This mesh was created to check if the jacobian was correctly implemented.

Edited by João Isler

Merge request reports

Approval is optional

Merged by Chris CantwellChris Cantwell 1 year ago (Jul 12, 2023 11:44am UTC)

Merge details

  • Changes merged into master with 142c6bd4 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jacques Xing
  • Jacques Xing
  • Jacques Xing
  • Chris Cantwell added 1 commit

    added 1 commit

    • 9c7c6623 - Generalise test fixture to support different types and files.

    Compare with previous version

  • Thibault Lestang added 2 commits

    added 2 commits

    • cd94f29a - fix: Call Configure() in bwdtrans test
    • 793cd901 - fix: test data location is assumed to be CMAKE_SOURCE_DIR

    Compare with previous version

  • Thibault Lestang
    Thibault Lestang @tlestang ·
    Maintainer

    Tests can be run with ctest --output-on-failure (or make test). Currently test_ipwrtbase fails.

  • João Isler added 1 commit

    added 1 commit

    Compare with previous version

  • João Isler added 11 commits

    added 11 commits

    • 64e729f3...cecde131 - 9 commits from branch master
    • 4fc5e88a - Update jacobian for deformed elements
    • 725e0b65 - Merge remote-tracking branch 'origin/master' into feature/IProdWRTBasePhysDerv

    Compare with previous version

  • Jacques Xing added 1 commit

    added 1 commit

    • 9702fc89 - Fix deformed/non-deformed blocks division bug

    Compare with previous version

  • Jacques Xing resolved all threads

    resolved all threads

  • Jacques Xing added 1 commit

    added 1 commit

    • 9279cfaf - Remove nested loop and fix interleave bug

    Compare with previous version

  • João Isler added 1 commit

    added 1 commit

    • d92b2f2b - Update values of the boost test case

    Compare with previous version

  • Jacques Xing
    Jacques Xing @CFD-Xing ·
    Maintainer

    @ccantwel This is ready to be merged. Tests have been included.

  • Jacques Xing added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Cantwell mentioned in commit 142c6bd4

    mentioned in commit 142c6bd4

  • Chris Cantwell merged

    merged

    • Please register or sign in to reply