opensourceleg icon indicating copy to clipboard operation
opensourceleg copied to clipboard

Dephy always logging

Open Katharine-Walters opened this issue 1 year ago • 2 comments

🐛 Bug Report

FlexSEA DebugLog is always being created regardless of the value of the dephy_log optional argument in the add_joint function.

🔬 How To Reproduce

Setting dephy_log = True and dephy_log = False results in the same behavior

Code sample

self.dev = OpenSourceLeg(frequency=1000, file_name="MBLUEankle_left_actuator_log.log") self.dev.add_joint(name="ankle", gear_ratio=9, port=self.actuator_port, dephy_log=False)

Environment

RPi 5 Python 3.11.2

Screenshots

Screenshot 2024-06-21 at 12 18 17 PM

📈 Expected behavior

dephy_log = False does not create DebugLog file

Katharine-Walters avatar Jun 21 '24 16:06 Katharine-Walters

Hello @Katharine-Walters, thank you for your interest in our work!

If this is a bug report, please provide screenshots and minimum viable code to reproduce your issue, otherwise we can not help you.

github-actions[bot] avatar Jun 21 '24 16:06 github-actions[bot]

Upon further investigation, I have discovered a few things: -The "dephy_log" bool is passed to the "debug" optional argument in the FlexSEA constructor. Is it unclear from the documentation what this controls, and I was unable to discern a change in behavior/logging when the bool was set to True or False. -I was able to disable the dephy logging by setting the debug_level=6, which disables the logging in the FlexSEA constructor.

I recommend implementing the following behavior: -Set default debug_level=4, which is the default logging level in the FlexSEA constructor. -When dephy_log = False, set debug_level=6, which will disable Dephy logging entirely.

Katharine-Walters avatar Jun 25 '24 21:06 Katharine-Walters

@all-contributors add @Katharine-Walters as a contributor for bug report

senthurayyappan avatar Jun 04 '25 16:06 senthurayyappan

@senthurayyappan

I've put up a pull request to add @Katharine-Walters! :tada:

allcontributors[bot] avatar Jun 04 '25 16:06 allcontributors[bot]