Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add --log (logging conf) option to main command#15

Merged
dwhswenson merged 2 commits into
openpathsampling:masteropenpathsampling/openpathsampling-cli:masterfrom
dwhswenson:loggingdwhswenson/openpathsampling-cli:loggingCopy head branch name to clipboard
Feb 27, 2020
Merged

Add --log (logging conf) option to main command#15
dwhswenson merged 2 commits into
openpathsampling:masteropenpathsampling/openpathsampling-cli:masterfrom
dwhswenson:loggingdwhswenson/openpathsampling-cli:loggingCopy head branch name to clipboard

Conversation

@dwhswenson

Copy link
Copy Markdown
Member

This will add the --log option, which takes a Python logging configuration file, to the main command.

Usage would be like this:

openpathsampling --log logging.conf pathsampling setup.nc -o output.nc -n 1000

Note that the --log option comes before the command.

@dwhswenson dwhswenson changed the title [WIP] Add --log (logging conf) option to main command Add --log (logging conf) option to main command Feb 25, 2020
@dwhswenson

Copy link
Copy Markdown
Member Author

I'll leave this up for comment or review for at least 24 hours, merging no earlier than Wednesday 26 February.

@dwhswenson dwhswenson merged commit 8a830a6 into openpathsampling:master Feb 27, 2020
@dwhswenson dwhswenson deleted the logging branch February 27, 2020 18:23
@dwhswenson dwhswenson added the enhancement New feature or request label Mar 2, 2020
This was referenced Apr 7, 2020
dwhswenson added a commit to OpenFreeEnergy/openfe that referenced this pull request Nov 2, 2022
A lot of this was taken from the OpenPathSampling CLI.

See openpathsampling/openpathsampling-cli#15
richardjgowers pushed a commit to OpenFreeEnergy/openfe that referenced this pull request Dec 6, 2022
A lot of this was taken from the OpenPathSampling CLI.

See openpathsampling/openpathsampling-cli#15
richardjgowers pushed a commit to OpenFreeEnergy/openfe that referenced this pull request Dec 6, 2022
A lot of this was taken from the OpenPathSampling CLI.

See openpathsampling/openpathsampling-cli#15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.