-
Notifications
You must be signed in to change notification settings - Fork 849
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
Unsteady CHT #854
Unsteady CHT #854
Changes from 30 commits
708121a
9d4e408
02eea23
f35f260
18af1de
7ce063e
34aa908
da8a691
dbc8ff7
9ce9811
4a6344c
64a170d
18fa12a
b8dd47c
36e4291
60358b6
725ceee
b8f571e
008c35a
4cebf34
382e82f
772756a
8e27cf8
4606456
d103c4a
7061830
0e4f584
6702690
99f5284
c3c114c
0796772
cac5f03
fd49a59
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -682,7 +682,8 @@ class CFEMFluidIteration : public CFluidIteration { | |
* \brief Class for driving an iteration of the heat system. | ||
* \author T. Economon | ||
*/ | ||
class CHeatIteration : public CIteration { | ||
class CHeatIteration : public CFluidIteration { | ||
|
||
public: | ||
|
||
/*! | ||
|
@@ -696,22 +697,6 @@ class CHeatIteration : public CIteration { | |
*/ | ||
~CHeatIteration(void); | ||
|
||
/*! | ||
* \brief Preprocessing to prepare for an iteration of the physics. | ||
* \param[in] ??? - Description here. | ||
*/ | ||
void Preprocess(COutput *output, | ||
CIntegration ****integration, | ||
CGeometry ****geometry, | ||
CSolver *****solver, | ||
CNumerics ******numerics, | ||
CConfig **config, | ||
CSurfaceMovement **surface_movement, | ||
CVolumetricMovement ***grid_movement, | ||
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
|
||
/*! | ||
* \brief Perform a single iteration of the heat system. | ||
* \param[in] output - Pointer to the COutput class. | ||
|
@@ -737,7 +722,7 @@ class CHeatIteration : public CIteration { | |
unsigned short val_iInst); | ||
|
||
/*! | ||
* \brief Perform a single iteration of the wave system. | ||
* \brief Iterate the heat system for a number of Inner_Iter iterations. | ||
* \param[in] output - Pointer to the COutput class. | ||
* \param[in] integration - Container vector with all the integration methods. | ||
* \param[in] geometry - Geometrical definition of the problem. | ||
|
@@ -747,19 +732,18 @@ class CHeatIteration : public CIteration { | |
* \param[in] surface_movement - Surface movement classes of the problem. | ||
* \param[in] grid_movement - Volume grid movement classes of the problem. | ||
* \param[in] FFDBox - FFD FFDBoxes of the problem. | ||
* \param[in] val_iZone - zone of the problem. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. not sure why you removed There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. They are missing everywhere. I think one should add those consistently. |
||
*/ | ||
void Solve(COutput *output, | ||
CIntegration ****integration, | ||
CGeometry ****geometry, | ||
CSolver *****solver, | ||
CNumerics ******numerics, | ||
CConfig **config, | ||
CSurfaceMovement **surface_movement, | ||
CVolumetricMovement ***grid_movement, | ||
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
void Solve(COutput *output, | ||
CIntegration ****integration, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is one space out of alignment There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To me it looks good and it does not say 'Outdated' 😕 (concerning @pcarruscag 's comment) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. void is 3 spaces from the margin 🤓 |
||
CGeometry ****geometry, | ||
CSolver *****solver, | ||
CNumerics ******numerics, | ||
CConfig **config, | ||
CSurfaceMovement **surface_movement, | ||
CVolumetricMovement ***grid_movement, | ||
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
|
||
/*! | ||
* \brief Updates the containers for the heat system. | ||
|
@@ -776,41 +760,6 @@ class CHeatIteration : public CIteration { | |
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
|
||
/*! | ||
* \brief Monitors the convergence and other metrics for the heat system. | ||
* \param[in] ??? - Description here. | ||
*/ | ||
bool Monitor(COutput *output, | ||
CIntegration ****integration, | ||
CGeometry ****geometry, | ||
CSolver *****solver, | ||
CNumerics ******numerics, | ||
CConfig **config, | ||
CSurfaceMovement **surface_movement, | ||
CVolumetricMovement ***grid_movement, | ||
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
|
||
/*! | ||
* \brief Postprocess ???. | ||
* \param[in] solver - Container vector with all the solutions. | ||
* \param[in] geometry - Geometrical definition of the problem. | ||
* \param[in] config - Definition of the particular problem. | ||
*/ | ||
void Postprocess(COutput *output, | ||
CIntegration ****integration, | ||
CGeometry ****geometry, | ||
CSolver *****solver, | ||
CNumerics ******numerics, | ||
CConfig **config, | ||
CSurfaceMovement **surface_movement, | ||
CVolumetricMovement ***grid_movement, | ||
CFreeFormDefBox*** FFDBox, | ||
unsigned short val_iZone, | ||
unsigned short val_iInst); | ||
|
||
}; | ||
|
||
/*! | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
/*! | ||
* \file CHeatSolverFVM.hpp | ||
* \brief Headers of the CHeatSolverFVM class | ||
* \author O. Burghardt | ||
* \file CHeatSolver.hpp | ||
* \brief Headers of the CHeatSolver class | ||
* \author F. Palacios, T. Economon | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unimportant ... but why did you change the file author? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's a very minor correction. Don't know why, maybe for consistency. There are no real guidelines for that anyway. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. "If you wrote everything you are the author" than seems to be the rule no? |
||
* \version 7.0.1 "Blackbird" | ||
* | ||
* SU2 Project Website: https://su2code.github.io | ||
|
@@ -28,14 +28,15 @@ | |
#pragma once | ||
|
||
#include "CSolver.hpp" | ||
#include "../variables/CHeatFVMVariable.hpp" | ||
#include "../variables/CHeatVariable.hpp" | ||
|
||
/*! \class CHeatSolverFVM | ||
* \brief Main class for defining the finite-volume heat solver. | ||
* \author O. Burghardt | ||
* \date January 19, 2018. | ||
/*! | ||
* \class CHeatSolver | ||
* \brief Main class for defining the finite-volume heat solver. | ||
* \author O. Burghardt | ||
* \version 7.0.1 "Blackbird" | ||
*/ | ||
class CHeatSolverFVM final : public CSolver { | ||
class CHeatSolver final : public CSolver { | ||
protected: | ||
unsigned short nVarFlow, nMarker, CurrentMesh; | ||
su2double **HeatFlux, *HeatFlux_per_Marker, *Surface_HF, Total_HeatFlux, AllBound_HeatFlux, | ||
|
@@ -44,7 +45,7 @@ class CHeatSolverFVM final : public CSolver { | |
*Surface_Areas, Total_HeatFlux_Areas, Total_HeatFlux_Areas_Monitor; | ||
su2double ***ConjugateVar, ***InterfaceVar; | ||
|
||
CHeatFVMVariable* nodes = nullptr; /*!< \brief The highest level in the variable hierarchy this solver can safely use. */ | ||
CHeatVariable* nodes = nullptr; /*!< \brief The highest level in the variable hierarchy this solver can safely use. */ | ||
|
||
/*! | ||
* \brief Return nodes to allow CSolver::base_nodes to be set. | ||
|
@@ -56,17 +57,17 @@ class CHeatSolverFVM final : public CSolver { | |
/*! | ||
* \brief Constructor of the class. | ||
*/ | ||
CHeatSolverFVM(void); | ||
CHeatSolver(void); | ||
|
||
/*! | ||
* \brief Constructor of the class. | ||
*/ | ||
CHeatSolverFVM(CGeometry *geometry, CConfig *config, unsigned short iMesh); | ||
CHeatSolver(CGeometry *geometry, CConfig *config, unsigned short iMesh); | ||
|
||
/*! | ||
* \brief Destructor of the class. | ||
*/ | ||
virtual ~CHeatSolverFVM(void); | ||
virtual ~CHeatSolver(void); | ||
|
||
/*! | ||
* \brief Restart residual and compute gradients. | ||
|
@@ -380,4 +381,4 @@ class CHeatSolverFVM final : public CSolver { | |
*/ | ||
inline su2double GetHeatFlux(unsigned short val_marker, unsigned long val_vertex) const override { return HeatFlux[val_marker][val_vertex]; } | ||
|
||
}; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whether typo