Skip to content

Reduce logging level#126

Merged
AadityaRavindran merged 2 commits into
masterfrom
aravindran/reduce-logging-level
Apr 27, 2026
Merged

Reduce logging level#126
AadityaRavindran merged 2 commits into
masterfrom
aravindran/reduce-logging-level

Conversation

@AadityaRavindran
Copy link
Copy Markdown
Contributor

Addresses #123 (comment)

Removes the excessive logging

INFO] [catkin_virtualenv]: grep -l -r -e ^#!.*bin/\(env \)\?\(python\|pypy\|ipy\|jython\) -e ^'''exec.*bin/\(python\|pypy\|ipy\|jython\) /home/aravindran/locus_dev/hotdog/devel/.private/nav_engine_scripts/share/nav_engine_scripts/venv/bin
[INFO] [catkin_virtualenv]: sed -i s-^#!.*bin/\(env \)\?\(python\|pypy\|ipy\|jython\)\"\?-#!/home/aravindran/locus_dev/hotdog/devel/\.private/nav_engine_scripts/share/nav_engine_scripts/venv/bin/python-;s-^'''exec'.*bin/\(python\|pypy\|ipy\|jython\)-'''exec' /home/aravindran/locus_dev/hotdog/devel/\.private/nav_engine_scripts/share/nav_engine_scripts/venv/bin/python- /home/aravindran/locus_dev/hotdog/devel/.private/nav_engine_scripts/share/nav_engine_scripts/venv/bin/distro
[INFO] [catkin_virtualenv]: sed -i s-^#!.*bin/\(env \)\?\(python\|pypy\|ipy\|jython\)\"\?-#!/home/aravindran/locus_dev/hotdog/devel/\.private/nav_engine_scripts/share/nav_engine_scripts/venv/bin/python-;s-^'''exec'.*bin/\(python\|pypy\|ipy\|jython\)-'''exec' 
...

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces default verbosity from INFO to WARN for catkin_virtualenv’s logging configuration, addressing excessive command/log output described in the linked discussion.

Changes:

  • Lower the default logging.basicConfig level from INFO to WARN in configure_logging().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread catkin_virtualenv/src/catkin_virtualenv/__init__.py Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@AadityaRavindran AadityaRavindran merged commit 9808b9d into master Apr 27, 2026
4 checks passed
@AadityaRavindran AadityaRavindran deleted the aravindran/reduce-logging-level branch April 27, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants