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

Chrome tracing #324

Open
wants to merge 17 commits into
base: unstable
Choose a base branch
from
Open

Chrome tracing #324

wants to merge 17 commits into from

Conversation

devinshende
Copy link
Contributor

Adds support for json files as output in order to use chrome tracing as a GUI to view the tau trace files.

@nchaimov
Copy link
Member

nchaimov commented Jul 9, 2019

Although the tests are passing when run locally, for some reason, on Travis, TAU is failing to build the tau_trace2json tool:

g++   -O2 -g -I../include  -w       -DTAU_GNU -DTAU_DOT_H_LESS_HEADERS -fPIC   tau_trace2json.cpp -c 
g++   -O2 -g -I../include  -w       -DTAU_GNU -DTAU_DOT_H_LESS_HEADERS -fPIC   tau_trace2json.o -o tau_trace2json -lm -L/home/travis/taucmdr-test/system/tau/tau-2.28.1/x86_64/lib -lTAU_traceinput-e1fcdaba 
/usr/bin/ld: cannot find -lTAU_traceinput-e1fcdaba
collect2: error: ld returned 1 exit status
make[1]: *** [tau_trace2json] Error 1

We need to figure out why that's not getting built correctly in Travis.

@zbeekman
Copy link
Member

I wonder if there is a dependency which is installed on the local system that's missing on Travis? I'll try building locally here to see if I can reproduce this.

@zbeekman
Copy link
Member

@nchaimov what should we do with this PR? Should I merge latest unstable and try testing with GitHub actions CI?

@nchaimov
Copy link
Member

Yeah, let's merge unstable into this branch and see if it works. It was failing due to problems building tau_trace2json but I think that was fixed in unstable with issue #342.

@codecov
Copy link

codecov bot commented Sep 17, 2020

Codecov Report

Merging #324 into unstable will decrease coverage by 13.56%.
The diff coverage is 43.62%.

Impacted file tree graph

@@              Coverage Diff              @@
##           unstable     #324       +/-   ##
=============================================
- Coverage     66.01%   52.45%   -13.57%     
=============================================
  Files            88       92        +4     
  Lines          7286     8049      +763     
  Branches       1223     1386      +163     
=============================================
- Hits           4810     4222      -588     
- Misses         2048     3406     +1358     
+ Partials        428      421        -7     
Flag Coverage Δ
#CI 52.45% <43.62%> (?)
#unittests 52.45% <43.62%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/taucmdr/cf/__init__.py 100.00% <ø> (ø)
packages/taucmdr/cf/objects.py 100.00% <ø> (ø)
...ckages/taucmdr/cf/software/libotf2_installation.py 87.50% <0.00%> (ø)
packages/taucmdr/cf/software/ompt_installation.py 63.15% <0.00%> (-7.44%) ⬇️
packages/taucmdr/cf/software/papi_installation.py 24.75% <0.00%> (-34.66%) ⬇️
packages/taucmdr/cf/storage/__init__.py 50.87% <0.00%> (ø)
packages/taucmdr/cf/storage/levels.py 85.00% <ø> (ø)
packages/taucmdr/cli/commands/__main__.py 29.68% <0.00%> (-2.06%) ⬇️
...kages/taucmdr/cli/commands/application/__init__.py 100.00% <ø> (ø)
packages/taucmdr/cli/commands/application/edit.py 50.00% <0.00%> (-11.30%) ⬇️
... and 117 more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants