Thanks Daniel. You are right - I suspect it will make very little difference in the jsfiddle example regardless of what happens on the back end.
We have a pretty varied use case for the API, across many of the charts and often with thousands of rows in the underlying data-set. In time we may move to just re-generating the charts as you suggest, but until such time that we can assess the impact and get a sense of the scope of changes required on our end I think we'll just downgrade to 45.1. Thanks again for your help. Oliver On Thursday, July 27, 2017 at 6:11:34 PM UTC-4, Daniel LaLiberte wrote: > > When you redraw the same chart, it does try to reduce some of the > recalculations, but we bail out pretty quickly for most changes, so there > is not usually much benefit. Go ahead and regenerate the chart and see > how it goes. > > On Thu, Jul 27, 2017 at 5:34 PM, Oliver G <[email protected] <javascript:> > > wrote: > >> Daniel - thanks for being so quick. >> >> I am also encountering the same problem with only calling `setOption(s)` >> (without updating the data-set). Is destroying/re-instantiating/re-drawing >> as a solution fairly expensive relative to the status quo (assuming that >> updating options currently doesn't re-render the entire chart)? >> >> Thanks again >> O >> >> On Thursday, July 27, 2017 at 5:05:36 PM UTC-4, Daniel LaLiberte wrote: >>> >>> Oliver, >>> >>> If figured out the problem, and I have a fix. The bad news is that no >>> workaround is possible other than recreating the chart or chartwrapper. >>> >>> On Thu, Jul 27, 2017 at 2:20 PM, Daniel LaLiberte <[email protected]> >>> wrote: >>> >>>> Oliver, >>>> >>>> Thanks for the demo of this problem. I see it happening as well, and >>>> it does appear to be a bug. No idea why it is happening or how you can >>>> work around it without recreating the ChartWrapper (or chart). >>>> >>>> Setting the options again in your changeChart function doesn't help. >>>> It doesn't seem to help to have an explicit viewWindow for the vAxis. >>>> >>>> Also, if you don't drag to zoom before clicking "update chart data", >>>> then the zoom does work in the new chart. So it is the first usage that >>>> seems to corrupt subsequent usages after a redraw. >>>> >>>> It is not related to the ChartWrapper since it happens with a >>>> ColumnChart by itself. >>>> >>>> On Thu, Jul 27, 2017 at 1:21 PM, Oliver G <[email protected]> wrote: >>>> >>>>> Daniel, we are encountering a new problem with 45.2 + ChartWrapper (it >>>>> may not be specific to ChartWrapper, however this is where its >>>>> manifesting). This issue does not show up in 45.1. >>>>> >>>>> Description: >>>>> It has to do with the functionality embodied in the "explorer"/zoom >>>>> option-set. In particular when a chart is freshly created everything >>>>> (zoom >>>>> functionality) works fine. However, once the chart data is updated via >>>>> wrapper.setDataTable/wrapper.draw, the zoom functionality does not >>>>> convey. >>>>> >>>>> Example: https://jsfiddle.net/9888wphs/ >>>>> >>>>> To reproduce: >>>>> Step 1: Use the drag-to-zoom to sub-select the original chart >>>>> (everything works fine). >>>>> Step 2. Click "Update chart data" (chart updates as expected). >>>>> Step 3. Use the drag-to-zoom to sub-select the updated chart (no >>>>> response). >>>>> >>>>> Thank you in advance. >>>>> >>>>> >>>>> On Thursday, June 15, 2017 at 3:09:38 PM UTC-4, Daniel LaLiberte wrote: >>>>> >>>>>> You can now load Google Charts version '45.2' by loading 'upcoming'. >>>>>> Please test this version and let us know if you experience any new >>>>>> problems. >>>>>> >>>>>> >>>>>> At this time, 'current' is still '45.1'. You can continue to load >>>>>> version '45' or '45.1' by specifying one of those versions. >>>>>> >>>>>> Anticipated production release date: June 28 >>>>>> On this date, loading 'current' or 'upcoming' will load '45.2'. >>>>>> >>>>>> This version includes a few bug fixes and many changes that involve >>>>>> internal refactoring, particularly in how options are processed, but >>>>>> there >>>>>> should be no visible changes. >>>>>> >>>>>> >>>>>> Summary of changes for v45.2: >>>>>> >>>>>> >>>>>> >>>>>> - >>>>>> >>>>>> Corecharts >>>>>> - >>>>>> >>>>>> Added tooltip.boxStyle options: e.g. >>>>>> >>>>>> 'tooltip': { >>>>>> >>>>>> 'boxStyle': { >>>>>> >>>>>> 'stroke': '#b2b2b2', 'strokeOpacity': 1, 'strokeWidth': 1.5, >>>>>> >>>>>> 'fill': 'white', 'fillOpacity': 1, >>>>>> >>>>>> 'shadow': { 'radius': 1, 'opacity': 0.2, 'xOffset': 0, >>>>>> 'yOffset': 2 } >>>>>> >>>>>> }} >>>>>> >>>>>> - >>>>>> >>>>>> Fix AreaChart legend items to use areaOpacity option. >>>>>> - >>>>>> >>>>>> Fix infinite loop when formatted domain values are >>>>>> (incorrectly) numeric. >>>>>> >>>>>> >>>>>> - >>>>>> >>>>>> Table >>>>>> - >>>>>> >>>>>> Fix paging buttons. >>>>>> - >>>>>> >>>>>> Add column type to header cell classlist. >>>>>> - >>>>>> >>>>>> Respect column property. >>>>>> - >>>>>> >>>>>> Fix accessibility labels when sorting is enabled. >>>>>> - >>>>>> >>>>>> Gantt >>>>>> - >>>>>> >>>>>> Fix selection object to include row index. >>>>>> >>>>>> >>>>>> -- >>>>>> Daniel LaLiberte >>>>>> <https://plus.google.com/100631381223468223275?prsrc=2> >>>>>> [email protected] 5CC, Cambridge MA >>>>>> >>>>> -- >>>>> You received this message because you are subscribed to the Google >>>>> Groups "Google Visualization API" group. >>>>> To unsubscribe from this group and stop receiving emails from it, send >>>>> an email to [email protected]. >>>>> To post to this group, send email to [email protected]. >>>>> Visit this group at >>>>> https://groups.google.com/group/google-visualization-api. >>>>> To view this discussion on the web visit >>>>> https://groups.google.com/d/msgid/google-visualization-api/aae0b389-a010-4332-b39c-91fe0cf04202%40googlegroups.com >>>>> >>>>> <https://groups.google.com/d/msgid/google-visualization-api/aae0b389-a010-4332-b39c-91fe0cf04202%40googlegroups.com?utm_medium=email&utm_source=footer> >>>>> . >>>>> >>>>> For more options, visit https://groups.google.com/d/optout. >>>>> >>>> >>>> >>>> >>>> -- >>>> Daniel LaLiberte >>>> <https://plus.google.com/100631381223468223275?prsrc=2> >>>> [email protected] 5CC, Cambridge MA >>>> >>> >>> >>> >>> -- >>> Daniel LaLiberte <https://plus.google.com/100631381223468223275?prsrc=2> >>> [email protected] 5CC, Cambridge MA >>> >> -- >> You received this message because you are subscribed to the Google Groups >> "Google Visualization API" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] >> <javascript:>. >> To post to this group, send email to [email protected] >> <javascript:>. >> Visit this group at >> https://groups.google.com/group/google-visualization-api. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/google-visualization-api/ca41c85f-e47d-4fe0-b300-f474dfb9927f%40googlegroups.com >> >> <https://groups.google.com/d/msgid/google-visualization-api/ca41c85f-e47d-4fe0-b300-f474dfb9927f%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> >> For more options, visit https://groups.google.com/d/optout. >> > > > > -- > Daniel LaLiberte <https://plus.google.com/100631381223468223275?prsrc=2> > [email protected] <javascript:> 5CC, Cambridge MA > -- You received this message because you are subscribed to the Google Groups "Google Visualization API" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/google-visualization-api. To view this discussion on the web visit https://groups.google.com/d/msgid/google-visualization-api/07b88062-4f5c-4932-9410-3405076c180e%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
