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

Dephy always logging #243

Closed
Katharine-Walters opened this issue Jun 21, 2024 · 2 comments
Closed

Dephy always logging #243

Katharine-Walters opened this issue Jun 21, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Katharine-Walters
Copy link

Katharine-Walters commented Jun 21, 2024

🐛 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 Katharine-Walters added the bug Something isn't working label Jun 21, 2024
Copy link

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.

@senthurayyappan senthurayyappan self-assigned this Jun 22, 2024
@Katharine-Walters
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

2 participants