-
Notifications
You must be signed in to change notification settings - Fork 194
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
yarplogger – new feature requested: export multiple .txt files with a single button #3077
Comments
|
Hi @randaz81, While I understand the purpose of raw log files, users often share them instead of the human-readable ones. To make them easier to understand, I suggest the following 2 options:
|
That's fine, you just need to open the provided .log with the yarplogger and inspect it from the gui... Or I can add a new button to iterate on all the logged processes and export a .txt for each of them. But I still think that the first solution is better. |
Hi @randaz81 Just a comment on this specific point:
While we can easily assume that everyone has a text editor installed, it may happen that someone who needs to consume the text log doesn't have the Therefore, if the text log exposes meaningful tags like |
IMO, the important piece of information is that the option with the an unexperienced user may easily use this one instead of the longest moreover, as I experienced recently, the text editor (be it notepad++ or Visual Code or else) is usually more searchable when looking at some particular error (like the |
@Ugo, that was not the purpose of a binary .log file. (maybe should be called .ycb?) It is not a matter of replacing a number with a string, the whole file is messed up, since it mixes the logs from multiple processes belogning to the same session. .log -> a session of logs (multiple processes, not human readable) This is my proposed solution:
Regarding:
I partially disagree on this, it's true that you can edit a xml file with notepad, but only if it is simple and not deeply nested. If you need to perform search/replace operations, ad-hoc xml processors perform the same job in a better way. In a similar way, the logger gui provides you tools to search things in the log much better than a notepad. For example, you can filter out all messages of a specific level (warning/errors) or containing specific words. So my recommendation is to use the gui, it was specifically designed to help you :-) |
@simeonedussoni big red button?!?? |
actually, the one corresponding to the "save" action. not the actual red one which stops everything |
Ah ok. Indeed, the caption of that button is "save session" and it is meant to take a binary snapshot of all logged processes, both running and terminated, as mentioned before, so you can see what was running when an error occurred. This is not a piece of information which can be easily extracted from a text file as you need to crosscheck details of logs coming from different processes. |
Issue renamed. |
Hi @randaz81
It wasn't clear to me that a For the remaining comments, I have the GUI and it's not a problem for me. However, I know other cases where some1 simply do not have YARP installed and sometimes are involved in debugging teams. Since people tend to share Having a way to export multiple files in text format would be of help. Footnotes |
Yes, I understood this. It would be helpful to have a similar button to export the .txt file for one or more processes as stated by @pattacini PS: BTW, I have YARP-eries only on the Linux Laptop I used for wrist debug etc, and most of the time I look at logfiles with Notepad++ or Studio Code on my workstation. By chance I always saved the logfiles with the menu procedure since I am usually interested in only the running process and not in the dead ones and I just discovered the save sessione when @sgiraz showed me the tricky message codes |
Is your feature request related to a problem? Please describe.
Attaching .log files from
yarplogger
helps diagnose issues inicub-tech-support
(e.g., robotology/icub-tech-support#1742 (comment)). However, deciphering them is tricky due to numerical log levels that appear when the log is exported using theSave to file
button (⬇️). For better readability, it would be great to export logs with text-based levels (DEBUG, INFO, WARNING, etc..).Describe the solution you'd like
As you can see to find this error I have to use a text editor with regex features looking for
5\n
in case I want to find the errors.It would be nice, especially for maintainers technicians, to have the
ERROR
string instead of the numeric value (as it happens for the log of theyarprobotinterface
.Of course, this is the specific example for the error case, but we should be able to do the same for all the log levels
Describe alternatives you've considered
Without this feature, we have to use the regex
5\n
(where 5 corresponds to theERROR
).Additional context
Steps to reproduce:
log
->export current log to text file
file
->Save log session ⬇️
@Nicogene @martinaxgloria @simeonedussoni @pattacini
The text was updated successfully, but these errors were encountered: