plasma-umass / coz

Coz: Causal Profiling
Other
4.04k stars 160 forks source link

viewer: Missing refresh of UI #138

Open jesperpedersen opened 4 years ago

jesperpedersen commented 4 years ago

In case you toggle a progress point the UI will fail to update and some times switch to a canonical path - e.g. /path/to/file.c:1 instead of just file.c:1

Changing "Sort By" will update the UI correctly.

Sample programs

gcc -O0 -fno-omit-frame-pointer -gdwarf-4 -ldl -o linear linear.c
gcc -O0 -fno-omit-frame-pointer -gdwarf-4 -lpthread -ldl -o threads threads.c

Firefox 69, Fedora 30 x86_64.

linear.txt threads.txt

ccurtsinger commented 4 years ago

Could you upload the profile.coz file that exhibits this behavior in the viewer?

jesperpedersen commented 4 years ago

Its the output from the sample apps included linear.coz.txt threads.coz.txt

Lanzaa commented 3 months ago

This was fixed in PR #169, but is not available on the public github pages version.

My local viewr build on commit 4659e50 does not have this issue. Could someone update the github pages deployment? Maybe @emeryberger ?