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

Ron/issue 26 tlog issue bug #28

Merged
merged 4 commits into from
Apr 9, 2024

Conversation

ron003
Copy link
Collaborator

@ron003 ron003 commented Apr 3, 2024

The appending of a TraceStreamer& operator<< method seem to be incorrect.
When, for example, in listrev,
TLOG() << listrev::ProgressUpdate(ERS_HERE, get_name(), oss_prog); is called,
the result is a message is like an ers::log and ers::error messages concatenated:
2024-Apr-03 10:38:53,912 LOG [void dunedaq::listrev::RandomDataListGenerator::do_work(std::atomic&) at /home/ron/work/DUNEPrj/fddaq-v4.3.0-a9-1-ron/sourcecode/listrev/plugins/RandomDataListGenerator.cpp:166] 2024-Apr-03 10:38:53,912 ERROR [void dunedaq::listrev::RandomDataListGenerator::do_work(std::atomic&) at /home/ron/work/DUNEPrj/fddaq-v4.3.0-a9-1-ron/sourcecode/listrev/plugins/RandomDataListGenerator.cpp:166] Generated list #1 with contents {384, 887, 778, 916} and size 4. DAQModule: rdlg

With the soon to be PR, the message becomes, as it should be:
2024-Apr-03 10:51:24,421 LOG [void dunedaq::listrev::RandomDataListGenerator::do_work(std::atomic&) at /home/ron/work/DUNEPrj/fddaq-v4.3.0-a9-1-ron/sourcecode/listrev/plugins/RandomDataListGenerator.cpp:166] Generated list #1 with contents {384, 887, 778, 916} and size 4. DAQModule: rdlg

@ron003 ron003 added the bug Something isn't working label Apr 3, 2024
@ron003 ron003 requested a review from eflumerf April 3, 2024 17:55
@bieryAtFnal bieryAtFnal added the miscellaneous deliverable A change that is/will be part of a release but is not substantial enough to be a daq-deliverable label Apr 3, 2024
@bieryAtFnal bieryAtFnal changed the base branch from production/v4 to prep-release/fddaq-v4.4.0 April 3, 2024 20:43
@ron003 ron003 merged commit 43bd906 into prep-release/fddaq-v4.4.0 Apr 9, 2024
1 check passed
@ron003 ron003 deleted the ron/issue_26_TLOG_issue_bug_ branch April 9, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working miscellaneous deliverable A change that is/will be part of a release but is not substantial enough to be a daq-deliverable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants