daroczig / daroczig/logger

log_warnings() etc. not compatible with profvis

Open
#70 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
HTML
Stars
323
Forks
46
Avg merge
1d 21h
Merged PRs (30d)
1

Description

Thanks for this great package! As a python native dabbling with R, it's great to find something so ergonomic for logging. This isn't really an issue _per se_, but more of an FYI that if you try to run code with one of the `log_warnings()`, `log_errors()`, etc. functions active from inside of a `profvis` profiling session, you'll get the following error:

```
Error in globalCallingHandlers(warning = function(m) { :
should not be called with handlers on the stack
Calls: profvis ... ..stacktraceon.. -> -> globalCallingHandlers
Execution halted
```

It's probably not urgent to find a workaround, but a note in the documentation might be helpful for others who come across this as a surprise.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.