Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
Gaudi
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Iterations
Requirements
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Test cases
Artifacts
Deploy
Releases
Container Registry
Model registry
Operate
Environments
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Issue analytics
Insights
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Gaudi
Gaudi
Commits
d51ff11c
Commit
d51ff11c
authored
4 years ago
by
Illya Shapoval
Browse files
Options
Downloads
Patches
Plain Diff
prefer passing const ref to EventSlot when dumping in PrecedenceSvc
parent
f79f7a2c
No related branches found
Branches containing commit
No related tags found
Tags containing commit
1 merge request
!1189
Extend the task precedence tracer for dumping dynamic graphs (video demo)
Changes
3
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
GaudiHive/src/IPrecedenceSvc.h
+2
-2
2 additions, 2 deletions
GaudiHive/src/IPrecedenceSvc.h
GaudiHive/src/PrecedenceSvc.cpp
+3
-3
3 additions, 3 deletions
GaudiHive/src/PrecedenceSvc.cpp
GaudiHive/src/PrecedenceSvc.h
+2
-2
2 additions, 2 deletions
GaudiHive/src/PrecedenceSvc.h
with
7 additions
and
7 deletions
GaudiHive/src/IPrecedenceSvc.h
+
2
−
2
View file @
d51ff11c
...
...
@@ -57,9 +57,9 @@ public:
virtual
const
std
::
string
printState
(
EventSlot
&
)
const
=
0
;
/// Dump precedence rules
virtual
void
dumpPrecedenceRules
(
EventSlot
&
)
=
0
;
virtual
void
dumpPrecedenceRules
(
const
EventSlot
&
)
=
0
;
/// Dump precedence trace
virtual
void
dumpPrecedenceTrace
(
EventSlot
&
)
=
0
;
virtual
void
dumpPrecedenceTrace
(
const
EventSlot
&
)
=
0
;
};
#endif
/* GAUDIHIVE_IPRECEDENCESVC_H_ */
This diff is collapsed.
Click to expand it.
GaudiHive/src/PrecedenceSvc.cpp
+
3
−
3
View file @
d51ff11c
...
...
@@ -8,11 +8,11 @@
* granted to it by virtue of its status as an Intergovernmental Organization *
* or submit itself to any jurisdiction. *
\***********************************************************************************/
#include
"PrecedenceSvc.h"
#include
"EventSlot.h"
#include
"PRGraph/Visitors/Promoters.h"
#include
"PRGraph/Visitors/Rankers.h"
#include
"PRGraph/Visitors/Validators.h"
#include
"PrecedenceSvc.h"
#include
<Gaudi/Algorithm.h>
#include
<Gaudi/Sequence.h>
...
...
@@ -307,7 +307,7 @@ const std::string PrecedenceSvc::printState( EventSlot& slot ) const {
}
// ============================================================================
void
PrecedenceSvc
::
dumpPrecedenceRules
(
EventSlot
&
slot
)
{
void
PrecedenceSvc
::
dumpPrecedenceRules
(
const
EventSlot
&
slot
)
{
if
(
!
m_dumpPrecRules
)
{
warning
()
<<
"To trace temporal and topological aspects of execution flow, "
...
...
@@ -333,7 +333,7 @@ void PrecedenceSvc::dumpPrecedenceRules( EventSlot& slot ) {
}
// ============================================================================
void
PrecedenceSvc
::
dumpPrecedenceTrace
(
EventSlot
&
slot
)
{
void
PrecedenceSvc
::
dumpPrecedenceTrace
(
const
EventSlot
&
slot
)
{
if
(
!
m_dumpPrecTrace
)
{
warning
()
<<
"To trace task precedence patterns, set DumpPrecedenceTrace "
...
...
This diff is collapsed.
Click to expand it.
GaudiHive/src/PrecedenceSvc.h
+
2
−
2
View file @
d51ff11c
...
...
@@ -64,10 +64,10 @@ public:
/// Dump precedence rules (available only in DEBUG mode, and must be enabled
/// with the corresponding service property)
void
dumpPrecedenceRules
(
EventSlot
&
)
override
;
void
dumpPrecedenceRules
(
const
EventSlot
&
)
override
;
/// Dump precedence trace (available only in DEBUG mode, and must be enabled
/// with the corresponding service property)
void
dumpPrecedenceTrace
(
EventSlot
&
)
override
;
void
dumpPrecedenceTrace
(
const
EventSlot
&
)
override
;
/// Precedence rules accessor
const
concurrency
::
PrecedenceRulesGraph
*
getRules
()
const
{
return
&
m_PRGraph
;
};
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment