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

Data view fails to display leaf nodes #84

Closed
wch opened this issue Jul 27, 2017 · 1 comment
Closed

Data view fails to display leaf nodes #84

wch opened this issue Jul 27, 2017 · 1 comment

Comments

@wch
Copy link
Member

wch commented Jul 27, 2017

(This is related to #83.) For example, see http://rpubs.com/wch/294546

In the flame graph, you can see that there are seven time samples. All of the samples have runif on top of the call stack, as can be seen in the flame graph. In the data viewer, however, there are no entries for runif.

Here's the code. If you run it and there happens to be a <GC> above a runif in the flame graph, then the runif will show up in the data viewer (but the <GC> won't).

profvis({
  fa <- function() {
    fb()
  }
  
  fb <- function() {
    runif(1e6)
  }

  fa()
  
  fb()
})

@javierluraschi Was this behavior intentional?

@javierluraschi
Copy link
Contributor

@wch not intentional, from what I remember. BTW. Just noticed that switching between horizontal and vertical states is introducing jitter in the UI, not sure if this is already being tracked.

@wch wch closed this as completed in c92b487 Aug 17, 2017
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

No branches or pull requests

2 participants