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

Make logging work in case of multiple child processes (e.g. via currently used joblib.Parallel) #1495

Open
yarikoptic opened this issue Sep 5, 2024 · 0 comments
Labels

Comments

@yarikoptic
Copy link
Member

Encountered while troubleshooting

If joblib does not implement it, we might want/need to e.g. via approach described in https://docs.python.org/3/howto/logging-cookbook.html#logging-to-a-single-file-from-multiple-processes . For now, in our single use in organize, I would just RF to return desired for logging side effects (error).

@yarikoptic yarikoptic added the UX label Sep 5, 2024
yarikoptic added a commit that referenced this issue Sep 5, 2024
joblib does not setup automagically some kind of logging for Parallel.
Filed dedicated to possibly see it implemented

- #1495

For the sake of current use case (e.g. troubleshooting
#1494) it should largely suffice to
return and log information about exception which was raised while loading
metadata.  This is what is done in this PR and while using buggy hdmf we do get
nice logging in the log file at DEBUG level.  No attempts were made to reduce
possibly a flood of duplicate log messages since per file metadata would have
unique values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant