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

Feature/weighted mean #1025

Closed

Conversation

Benoit-DunandLaisin
Copy link

From #838

Hi,
Here is an implementation of the weighted mean charts value.

Feedbacks are welcome.

Regards,

@ruchitgandhi
Copy link

We add a numeric field for the weight and quantity field right? Can we also specify a query to select the total of some quantities for the weight field.

Like suppose I have field A(which is non-numeric) which has an associated field B(numeric). Can I specify something that :-
weight field = B only when A = "something" ?
quantity field = B when A = "Something else"

And what exactly is the difference in "weight , value , quantity" fields ?

@w33ble w33ble closed this Oct 6, 2014
@spalger
Copy link
Contributor

spalger commented Oct 6, 2014

Woops! Sorry about that! We recently replaced the master branch with Kibana 4. This action force closed all of the old pull requests against master. We will be reviewing these on a case-by-case basis and creating new tickets as necessary.

The good news is that many long requested features can be found in Kibana 4, and we're being entirely open about our roadmap. Check out the roadmap tickets (which we're still filling in) here.

If you're looking for the old Kibana 3 code you can find it here.

@w33ble
Copy link
Contributor

w33ble commented Oct 7, 2014

We're focusing on Kibana 4 going forward, which is an enitirely new codebase. I'm leaving this PR unmerged, but keeping the issue open so it can be addressed in Kibana 4.

@baselsalam
Copy link

+1.

w33ble added a commit to w33ble/kibana that referenced this pull request Sep 13, 2018
* feat: add lHeight alias

lineHeight just makes more sense to me

* fix: prevent font lineHeight from being too small

* fix: remove default line height, fall back to 1

according to mdn, a line-height of just a number is preferred, and when used, acts as a multiplier. see https://developer.mozilla.org/en-US/docs/Web/CSS/line-height

* chore: update line height test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants