-
Notifications
You must be signed in to change notification settings - Fork 239
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
Output primary variables on arbitrary meshes. #2299
Conversation
ProcessLib/Output/Output.cpp
Outdated
@@ -145,9 +158,9 @@ void Output::doOutputAlways(Process const& process, | |||
time_output.start(); | |||
|
|||
// Need to add variables of process to vtu even no output takes place. | |||
processOutputData(t, x, process.getMesh(), process.getDOFTable(process_id), | |||
processOutputData(t, x, *_meshes[0], process.getDOFTable(process_id), |
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.
discussion needed.
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.
Looks good except that is commented by Dima.
ProcessLib/Output/Output.cpp
Outdated
|
||
DBUG("output to %s", mesh_output_file_path.c_str()); | ||
|
||
//ProcessData* process_data = findProcessData(process, process_id); |
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.
Do you still need these two lines?
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.
Add TODO for pvd and then a note for merging the bulk and arbitrary mesh output
ProcessLib/Output/Output.cpp
Outdated
|
||
processOutputData(t, x, mesh, *mesh_dof_table, | ||
process.getProcessVariables(process_id), | ||
process.getSecondaryVariables(), false, |
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.
false
-> false /* no secondary vars output*/
ProcessLib/Output/Output.cpp
Outdated
// For the staggered scheme for the coupling, only the last process, | ||
// which gives the latest solution within a coupling loop, is allowed to | ||
// make output. | ||
if (!(process_id == |
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.
predicate
ProcessLib/Output/Output.cpp
Outdated
|
||
DBUG("output to %s", mesh_output_file_path.c_str()); | ||
|
||
//ProcessData* process_data = findProcessData(process, process_id); |
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.
Add TODO for pvd and then a note for merging the bulk and arbitrary mesh output
The functionality of the ProcessVariable isn't needed here.
b5f16bf
to
d4a959a
Compare
OpenGeoSys development has been moved to GitLab. |
As titled.