Skip to content

Activate root logger if not configured yet#403

Merged
ctk3b merged 1 commit intoshirtsgroup:masterfrom
jotelha:2026-01-15-logginf-fix
Jan 15, 2026
Merged

Activate root logger if not configured yet#403
ctk3b merged 1 commit intoshirtsgroup:masterfrom
jotelha:2026-01-15-logginf-fix

Conversation

@jotelha
Copy link
Contributor

@jotelha jotelha commented Jan 15, 2026

This is necessary for CLI applications to actually print logging information to stdout.

Without these two lines, the logger of intermol does not print anything to the command line when the application is called from the command line, even if specifically requested with the --verbose flag.

As an example, I have been using intermol with this command:

intermol-convert --gro_in default.gro system.top --lammps --oname converted --energy --lmppath $(which lmp) --gropath $GMXBIN --verbose

and would only see a single line printed directly with a print statement in the convert.py file.

{'lammps': 12057.699850550001}

…application to actually print logging information to stdout
@ctk3b ctk3b merged commit e40c079 into shirtsgroup:master Jan 15, 2026
0 of 5 checks passed
@ctk3b
Copy link
Member

ctk3b commented Jan 15, 2026

Thanks for the fix!

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.

2 participants