You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2019/05/27 20:27:09 UTC
[GitHub] [incubator-superset] villebro commented on issue #7379: Remove the
use of Pandas' iloc() in WorldMapViz
villebro commented on issue #7379: Remove the use of Pandas' iloc() in WorldMapViz
URL: https://github.com/apache/incubator-superset/pull/7379#issuecomment-496299757
@elukey I was able to replicate the problem on master, i.e. when choosing the same metric twice (color and bubble size), the error `Too many indexers` would show up. After applying your fix the problem went away without causing adverse side-effects that I could tell. If we want to be on the safe side, we could check if `df[metric]` is a `DataFrame` and call `iloc` on that. But I agree that there shouldn't be duplicate columns anymore, so it might very well just introduce unnecessary clutter into the codebase. So all in all LGTM.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org