Skip to content
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

[CLI] add --unbuffered-std-out option to deactivate buffer for cout #1514

Merged
merged 1 commit into from
Nov 4, 2016

Conversation

norihiro-w
Copy link
Collaborator

as titled. This is helpful for those who saves/forwards stadard output to a file and wants to trace real-time output from ogs.

Copy link
Member

@endJunction endJunction left a comment

Choose a reason for hiding this comment

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

Seems logog has no such functionality indeed.

@chleh
Copy link
Collaborator

chleh commented Nov 3, 2016

Looks good. But wouldn't it be better to automatically flush stdout once after every iteration/timestep?

@norihiro-w
Copy link
Collaborator Author

@chleh automatic flushing should be avoided in some simulations particularly on cluster computers, because it can cause high load on the system File I/O.

@TomFischer
Copy link
Member

automatic flushing should be avoided in some simulations particularly on cluster computers, because it can cause high load on the system File I/O.

Maybe this can be added as documentation into the command line option?

Copy link
Member

@TomFischer TomFischer left a comment

Choose a reason for hiding this comment

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

👍

@endJunction endJunction merged commit 13fb9c5 into ufz:master Nov 4, 2016
@norihiro-w norihiro-w deleted the unbuffer-cout branch November 7, 2016 05:57
@ogsbot
Copy link
Member

ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

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.

5 participants