# Bug Report: Power Table configureCell() **URL:** \ **Category:** Ignition\ **Tags:** ignition79, ignition81, vision\ **Created:** [September 24, 2021, 2:20pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315 "2021-09-24T14:20:55Z")\ **Posts on this page:** 20\ **Page:** 1
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 2:20pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/1 "2021-09-24T14:20:55Z")
There’s a bug in the `configureCell()` extension function on the Power Table component in Vision. I tested on 8.1 and 7.9, both of which have the bug. When selecting a row from a power table, `configureCell()` does not execute for all cells. Instead, the cells it executes on depend on the index of the selected row. There appears to be a logical bug in how the cells are looped over. To reproduce the bug: 1. Create a Power Table and populate it with 5 values in one column of data. The bug is present regardless of the number of columns or rows, but it’s easier to test with less data. 2. Set the Power Table’s configureCell() function to: ```auto def configureCell(...): print rowIndex, self.selectedRow ``` 1. Run the component and select various rows. In the debugging console you’ll see that the rows being processed by `configureCell()` are not consistent and depend on the row selected in the UI. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 2:28pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/2 "2021-09-24T14:28:05Z")
No, configureCell is called any time the cell needs to be _ **displayed** _ or refreshed. Scrolling, selecting, and data updates can all trigger this call, and it will only be called as needed. Cells out of sight due to scrolling may not be called at all. Rethink whatever you are trying to do in configureCell that doesn’t fit the display paradigm. Certainly do not attempt to change any state in the table there. Not a bug (in Ignition). ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 2:40pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/3 "2021-09-24T14:40:50Z")
Thanks for the insight, however, what you are describing doesn’t reflect the actual behavior of the component. It’s also not true that `configureCell()` “will only be called as needed”. If you actually test it, you’ll see that it executes when it isn’t necessary and the execution behavior is inconsistent depending on the row you select and which row was previously selected. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 2:47pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/4 "2021-09-24T14:47:58Z")
Yeah, well, it’s Swing underneath. The repaint operation is the primary driver, and “dirty” display rectangles can merge/overlap in odd ways. So, it should read “called as needed as determined by Swing”. If you are really curious, you can construct and log a fake java exception to see all of the call chains. Have fun. Still not a bug (in Ignition). Attempting to do anything other that supplying display attributes for the requested cell is a recipe for grief. ---
**Author:** ![jlandwerlen](https://avatars.discourse-cdn.com/v4/letter/j/bc8723/32.png) [@jlandwerlen](https://forum.inductiveautomation.com/u/jlandwerlen)\ **Post date:** [September 24, 2021, 2:49pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/5 "2021-09-24T14:49:42Z")
> [@nunya.biz](#): > > I tested on 8.1 and 7.9, both of which have the bug. Really think a bug would be around that long? Possible, not likely though. Just because it doesn't act like you want, doesn't make it a bug. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 2:54pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/6 "2021-09-24T14:54:56Z")
> [@jlandwerlen](#): > > Really think a bug would be around that long? Bite your tongue! This one lasted from FactoryPMI days through v7.9.14/v8.0.6 : > [@ConfigureChart Extension Function Trigger](https://forum.inductiveautomation.com/t/configurechart-extension-function-trigger/30713/5): > > Follow-up: The memory leak triggered by the Classic Chart update (anything that would trigger configureChart) was fixed in v8.0.6 but hasn’t yet made it into any version of v7.9. v7.9.13 was released shortly after v8.0.6, but this fix apparently did not make the cut. v7.9.14 will likely have it, and is expected in late April. ---
**Author:** ![jlandwerlen](https://avatars.discourse-cdn.com/v4/letter/j/bc8723/32.png) [@jlandwerlen](https://forum.inductiveautomation.com/u/jlandwerlen)\ **Post date:** [September 24, 2021, 2:55pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/7 "2021-09-24T14:55:14Z")
That’s why I said possible ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 2:57pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/8 "2021-09-24T14:57:30Z")
The function is not running for the rows that are actually displayed, instead, it runs for all rows between the row you select and the row previously selected. This is a bug. I think @pturmel’s explanation helps narrow down what’s happening. There appears to be a logical bug in determining which rows are displayed and therefore which need to be updated. ---
**Author:** ![jlandwerlen](https://avatars.discourse-cdn.com/v4/letter/j/bc8723/32.png) [@jlandwerlen](https://forum.inductiveautomation.com/u/jlandwerlen)\ **Post date:** [September 24, 2021, 2:58pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/9 "2021-09-24T14:58:53Z")
If you truly think it’s a bug, contact IA support and they will issue you a ticket and then you can show them. [https://support.inductiveautomation.com/hc/en-us](https://support.inductiveautomation.com/hc/en-us) ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 3:01pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/10 "2021-09-24T15:01:07Z")
I guess the real question is why do you care if it is called more than “necessary” ? ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 3:05pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/11 "2021-09-24T15:05:09Z")
Good question. On large tables, it can cause serious performance issues. We’ve ran into this problem multiple times over the years with Ignition. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 3:10pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/12 "2021-09-24T15:10:27Z")
That usually means you are accessing stuff in `configureCell` that isn’t local to the component. Like `system.*` calls that go to the gateway. There should be custom properties on your table that pull in all ancillary data you need for display, so the `configureCell` event itself doesn’t have to go get it. (It is already there.) If your performance craters when you run a remote client over a satellite connection or worldwide VPN, you’ve probably made this kind of mistake. ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 3:17pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/13 "2021-09-24T15:17:31Z")
That’s good advice. With the extra executions, we’ve seen performance issues on large tables even when executing simple logic that doesn’t rely on anything external to the table. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 3:21pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/14 "2021-09-24T15:21:09Z")
If you have examples, post them here. Or ask support for help if it isn’t something you can share. ---
**Author:** ![paul-griffith](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/paul-griffith/32/37249_2.png) [@paul-griffith](https://forum.inductiveautomation.com/u/paul-griffith)\ **Post date:** [September 24, 2021, 4:01pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/15 "2021-09-24T16:01:28Z")
As Phil said, this is not an Ignition bug. You can easily demonstrate the same behavior in a standalone Swing project with a JideTable or JTable you create yourself; [`TableCellRenderer`](https://docs.oracle.com/en/java/javase/11/docs/api/java.desktop/javax/swing/table/TableCellRenderer.html) makes no guarantees about when it will be called, and the behavior of painting and repainting is non-trivial. It is absolutely true that on very large tables, even the minimal overhead of calling into Jython is going to add up. You can minimize that as much as possible by using custom properties well, as Phil recommended, but there is a limit. In theory, I would expect a fully custom Java component (authored through the module SDK) to perform at the same rendering task than the power table, because it wouldn’t have the additional Jython overhead - but obviously that’s a tall order to set up. ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 4:19pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/16 "2021-09-24T16:19:40Z")
To workaround this bug in Swing, I update the table data when `selectedRow` changes. `Power_Table.propertyChange()`: ```auto if event.propertyName == 'selectedRow': event.source.data = event.source.data ``` It isn’t pretty, but it works. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 4:21pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/17 "2021-09-24T16:21:21Z")
Some tips for superior performance on events that get called (a lot) by Swing: - Move all use of `class` and `def` to a project script module. - Make all ancillary data needed available in custom properties. Preferably using asynchronous bindings (Tag bindings and Query bindings) or property bindings. If a script is needed to collect ancillary data, use system.util.invokeAsynchronous() to do the time-consuming part off the GUI thread. - Avoid loops. If a lookup will be needed, post-process the corresponding ancillary data into a Python dictionary, and cache it in the component’s Swing “client properties”. The fast path in the event would use `d=self.getClientProperty(...)` then `d.get(someKey)` to perform the lookup. - Move as much code as practical into a project script module. ---
**Author:** ![pturmel](https://sea1.discourse-cdn.com/inductiveautomation/user_avatar/forum.inductiveautomation.com/pturmel/32/3295_2.png) [@pturmel](https://forum.inductiveautomation.com/u/pturmel)\ **Post date:** [September 24, 2021, 4:22pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/18 "2021-09-24T16:22:47Z")
> [@nunya.biz](#): > > It isn’t pretty, but it works. OMG! No! That makes configureCell get called _ **even more** _. ---
**Author:** ![nunya.biz](https://avatars.discourse-cdn.com/v4/letter/n/d2c977/32.png) [@nunya.biz](https://forum.inductiveautomation.com/u/nunya.biz)\ **Post date:** [September 24, 2021, 4:25pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/19 "2021-09-24T16:25:47Z")
👍 👍 I’ve surrendered to the pain ---
**Author:** ![jlandwerlen](https://avatars.discourse-cdn.com/v4/letter/j/bc8723/32.png) [@jlandwerlen](https://forum.inductiveautomation.com/u/jlandwerlen)\ **Post date:** [September 25, 2021, 2:34pm UTC](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315/20 "2021-09-25T14:34:11Z")
> [@pturmel](#): > > - Avoid loops. If a lookup will be needed, post-process the corresponding ancillary data into a Python dictionary, and cache it in the component's Swing "client properties". The fast path in the event would use `d=self.getClientProperty(...)` then `d.get(someKey)` to perform the lookup. Does anything exist in the manual, or elsewhere, that explains this in more detail? [Next page](https://forum.inductiveautomation.com/t/bug-report-power-table-configurecell/51315.md?page=2)