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

enable working with percentiles #40

Closed

Conversation

dulcineapena1
Copy link

Releted to #39
Add the hability to work with percentiles attributes values.

@CLAassistant
Copy link

CLAassistant commented Jul 3, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@glitton glitton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @dulcineapena1,
Thank you for your pull request and I apologize for the delay in my response.

I just checked your code, ran it and encountered a ReferenceError specifically

Uncaught ReferenceError: cells is not defined
at EditBoard.render

The above should be an easy fix on your part. However, I am still unsure by what you are trying to achieve. You are adding functionality that enables working with percentiles. Can you explain further what you expect to achieve? Are the percentiles the output of the board? Appreciate you taking the time to explain.

I

@dulcineapena1
Copy link
Author

Hello @glitton
Thanks for your answer. Unfortunately I could not replicate that error that you mention... I will appreciate if you could tell me more about where did you get the error...

Talking about the explanation that you ask me, I notice when I try to work with percentiles, I got a blank cell instead of the bullet or the value. For example if I select an event type SystemSample, attribute median(cpuSystemPercent), I will have a blank cell in my board; and in case I add a comparison less than 1, my entire board will crash with the message 'Oops! Something went wrong.' I mean, I would expect to have a bullet in my board, similar that the ones you get when you are working with another type of attribute.
I don't know if there is a good thing to have this, or maybe Neon board is not supposed to work with percentiles.

Thanks for your time!

@danielgolden
Copy link
Contributor

@glitton Any thoughts on @dulcineapena1's latest comment?

@danielgolden danielgolden linked an issue Aug 19, 2020 that may be closed by this pull request
@tangollama tangollama changed the base branch from master to main September 3, 2020 03:29
@jpvajda
Copy link

jpvajda commented Feb 23, 2021

This seems like a stale PR, so I'll close unless someone can confirm this is still being worked on.

@jpvajda jpvajda added question Further information is requested stale labels Feb 23, 2021
@dulcineapena1
Copy link
Author

dulcineapena1 commented Feb 23, 2021

This seems like a stale PR, so I'll close unless someone can confirm this is still being worked on.

@jpvajda
You can close this PR, I was working on it few months ago, but I am not longer on it because I couldn't find the requested error... I thought this could be a usefull functionality but maybe this is not that is completely necesary for now.
Thank you.

@jpvajda
Copy link

jpvajda commented Feb 24, 2021

@dulcineapena1 ok thanks for confirming.

@jpvajda jpvajda removed the question Further information is requested label Feb 24, 2021
@jpvajda jpvajda closed this Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not working with percentile attributes
5 participants