-
Notifications
You must be signed in to change notification settings - Fork 494
Write standalone --debug output to CSV file #15182
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
Changes from 14 commits
d1e320d
61a9e88
0a93b3a
d1456c5
408e624
5c23211
38568a6
0beca91
c01c80e
f09d2f0
fcfda95
52b738c
f63cc55
7005e1b
ee79d3f
0587c3a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,6 +18,7 @@ | |
| #include <cstddef> | ||
| #include <cstdio> | ||
| #include <cstring> | ||
| #include <fstream> | ||
| #include <string> | ||
| #include <memory> | ||
| #include <iosfwd> | ||
|
|
@@ -423,6 +424,17 @@ class GPUReconstruction | |
| void debugInit(); | ||
| void debugExit(); | ||
|
|
||
| struct debugWriter { | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In order to keep GPUReconstruction.h, which is included often, small, we could move this sub-class to GPUReconstructionCPU.h, since it is used only there.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done. Note that I now had to include GPUReconstructionCPU.h in GPUReconstructionDebug.cxx. This was not needed before. |
||
| debugWriter(std::string filenameCSV, bool markdown, uint32_t statNEvents); | ||
| void header(); | ||
| void row(char type, uint32_t count, std::string name, double gpu_time, double cpu_time, double total_time, std::size_t memSize, std::string nEventReport = ""); | ||
|
|
||
| private: | ||
| std::ofstream streamCSV; | ||
| bool mMarkdown; | ||
| uint32_t mStatNEvents; | ||
| }; | ||
|
|
||
| static GPUReconstruction* GPUReconstruction_Create_CPU(const GPUSettingsDeviceBackend& cfg); | ||
| }; | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove also the fstream here, which is no longer needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done