search

Esri / ember-cli-cedar

Ember addon for Esri's Cedar charting library
https://esri.github.io/ember-cli-cedar/#/charts/bar
Apache License 2.0
3 stars 2 forks source link

Stacked Chart example not rendering properly according to data #88

Closed chagevik closed 6 years ago

chagevik commented 6 years ago

https://esri.github.io/ember-cli-cedar/#/charts/bar-stacked

On this example, the values form the example service, do not seem to be rendering correctly. I would expect to see the following values based on queries to the service:

Jordan > "query": { "where": "City='Jordan'", "orderByFields": "Number_of_SUM DESC", "groupByFieldsForStatistics": "Type", "outStatistics": [{ "statisticType": "sum", "onStatisticField": "Number_of", "outStatisticFieldName": "Number_of_SUM" }] }

Results >> High School > 13 Middle School > 6 Elementary School > 1 ** Showing Correctly

Dewitt > "name": "Dewitt", "query": { "where": "City='Dewitt'", "orderByFields": "Number_of_SUM DESC", "groupByFieldsForStatistics": "Type", "outStatistics": [{ "statisticType": "sum", "onStatisticField": "Number_of", "outStatisticFieldName": "Number_of_SUM" }] }

Results >> Elementary School > 1 ** Showing values in High School that are note returned

Fayetteville > "query": { "where": "City='Fayetteville'", "orderByFields": "Number_of_SUM DESC", "groupByFieldsForStatistics": "Type", "outStatistics": [{ "statisticType": "sum", "onStatisticField": "Number_of", "outStatisticFieldName": "Number_of_SUM" }] }

Results >> High School > 8 Middle School > 0 Elementary School > 1 ** No values showing for High School or Middle School (looks like Dewitt took on these values instead)

It seems like the Dewitt category is taking on the values for the Fayetteville category since the query did not return anything for these stacked categories.

Does this chart require all categories shown to have values for all of the categories within the stacked bars in order to show correctly?

tomwayson commented 6 years ago

Good catch @cher7369!

We just fixed this bug in cedar https://github.com/Esri/cedar/pull/411 and will be cutting a release w/ that today, and then updating ember-cli-cedar. I will ping you when that is done.

chagevik commented 6 years ago

Great thanks Tom!

From: Tom Wayson notifications@github.com Sent: Friday, March 30, 2018 1:28 PM To: Esri/ember-cli-cedar ember-cli-cedar@noreply.github.com Cc: Cheryl Hagevik CHagevik@esri.com; Mention mention@noreply.github.com Subject: Re: [Esri/ember-cli-cedar] Stacked Chart example not rendering properly according to data (#88)

Good catch @cher7369https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_cher7369&d=DwMFaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=xnCSVirz6ai0VtfsJAP3Q6gvjm1qL7mccglNhlZDQj8&m=YLoKc0bmN7qsfIdH9RnYrjFa4c_pvFEyH9PZn9JW2mM&s=i_tVpqxcXiAF6QMPsVdC6jXYmxdsPXHGQ0ja10_kWEA&e=!

We just fixed this bug in cedar Esri/cedar#411https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_cedar_pull_411&d=DwMFaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=xnCSVirz6ai0VtfsJAP3Q6gvjm1qL7mccglNhlZDQj8&m=YLoKc0bmN7qsfIdH9RnYrjFa4c_pvFEyH9PZn9JW2mM&s=13ngvU2LWTNHy-_U4bbYHqGjv-uFJTb9u6XFkYOhLuU&e= and will be cutting a release w/ that today, and then updating ember-cli-cedar. I will ping you when that is done.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ember-2Dcli-2Dcedar_issues_88-23issuecomment-2D377576558&d=DwMFaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=xnCSVirz6ai0VtfsJAP3Q6gvjm1qL7mccglNhlZDQj8&m=YLoKc0bmN7qsfIdH9RnYrjFa4c_pvFEyH9PZn9JW2mM&s=u_twbYMQsZKwO8UDGx_a3Ht0UGBYK1jGe2g8MKsOZm4&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGpuO9nklKntmf1gYLiFEigDrHkoapT7ks5tjms1gaJpZM4TBzjh&d=DwMFaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=xnCSVirz6ai0VtfsJAP3Q6gvjm1qL7mccglNhlZDQj8&m=YLoKc0bmN7qsfIdH9RnYrjFa4c_pvFEyH9PZn9JW2mM&s=l_yICXcUm4W_wK3Hnp7PDQoKa8_6Sg7rvFaGEzq5lnY&e=.