[GitHub] daphne-jr commented on issue #4403: Test case not passing when callbacks are used in DateFilterControl_spec.jsx + a repeated test case

2018-02-12 Thread GitBox
daphne-jr commented on issue #4403: Test case not passing when callbacks are used in DateFilterControl_spec.jsx + a repeated test case URL: https://github.com/apache/incubator-superset/issues/4403#issuecomment-364896693 After trying for some time today, this is my crude solution. Hope it

[GitHub] THDA commented on issue #2436: Slow query in SQL Lab

2018-02-12 Thread GitBox
THDA commented on issue #2436: Slow query in SQL Lab URL: https://github.com/apache/incubator-superset/issues/2436#issuecomment-364883611 Confirmed here, this is dead slow and needs fixing, it even triggers a "this site does not respond" on ie. Turning on profiling, I see the following

[GitHub] pastarace commented on issue #2123: How to daemonize superset process

2018-02-12 Thread GitBox
pastarace commented on issue #2123: How to daemonize superset process URL: https://github.com/apache/incubator-superset/issues/2123#issuecomment-364893687 Hi running the script from @fuzaro I'm getting `/bin/sh: 1: gunicorn: not found` message, that's due to the virtualenv is not active,

[GitHub] romankierzkowski opened a new issue #4407: Error on SQL execution: "Could not broadcast input array from shape (26896) into shape (1,164)"

2018-02-12 Thread GitBox
romankierzkowski opened a new issue #4407: Error on SQL execution: "Could not broadcast input array from shape (26896) into shape (1,164)" URL: https://github.com/apache/incubator-superset/issues/4407 Make sure these boxes are checked before submitting your issue - thank you! - [x ]

[GitHub] mistercrunch closed pull request #4380: [error handling] 'Time Comparison' query returns no data

2018-02-12 Thread GitBox
mistercrunch closed pull request #4380: [error handling] 'Time Comparison' query returns no data URL: https://github.com/apache/incubator-superset/pull/4380 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] maver1ck commented on issue #4374: Very big visualizations' thumbnails size

2018-02-12 Thread GitBox
maver1ck commented on issue #4374: Very big visualizations' thumbnails size URL: https://github.com/apache/incubator-superset/issues/4374#issuecomment-365012342 ping @mistercrunch This is an automated message from the

[GitHub] andrewsali commented on issue #4398: database_access on [mydb] permissionview not created when database added

2018-02-12 Thread GitBox
andrewsali commented on issue #4398: database_access on [mydb] permissionview not created when database added URL: https://github.com/apache/incubator-superset/issues/4398#issuecomment-364984407 The following resolves this issue:

[GitHub] timifasubaa opened a new pull request #4408: Add link on how to get permission.

2018-02-12 Thread GitBox
timifasubaa opened a new pull request #4408: Add link on how to get permission. URL: https://github.com/apache/incubator-superset/pull/4408 With the access requests feature removed, this PR creates a way for users to be shown how to gain access to datasources. @john-bodley

[GitHub] timifasubaa commented on issue #4405: Disable user access request

2018-02-12 Thread GitBox
timifasubaa commented on issue #4405: Disable user access request URL: https://github.com/apache/incubator-superset/pull/4405#issuecomment-365092528 @mistercrunch Done. I left the access requests on by default. Is it better off by default?

[GitHub] timifasubaa commented on issue #4405: Disable user access request

2018-02-12 Thread GitBox
timifasubaa commented on issue #4405: Disable user access request URL: https://github.com/apache/incubator-superset/pull/4405#issuecomment-365092528 @mistercrunch Done This is an automated message from the Apache Git

[GitHub] mistercrunch commented on issue #4398: database_access on [mydb] permissionview not created when database added

2018-02-12 Thread GitBox
mistercrunch commented on issue #4398: database_access on [mydb] permissionview not created when database added URL: https://github.com/apache/incubator-superset/issues/4398#issuecomment-365085614 @andrewsali thank you for digging in, please open a PR!

[GitHub] mistercrunch commented on issue #4407: Error on SQL execution: "Could not broadcast input array from shape (26896) into shape (1,164)"

2018-02-12 Thread GitBox
mistercrunch commented on issue #4407: Error on SQL execution: "Could not broadcast input array from shape (26896) into shape (1,164)" URL: https://github.com/apache/incubator-superset/issues/4407#issuecomment-365091430 Which database engine are you querying here?

[GitHub] mistercrunch commented on issue #4374: Very big visualizations' thumbnails size

2018-02-12 Thread GitBox
mistercrunch commented on issue #4374: Very big visualizations' thumbnails size URL: https://github.com/apache/incubator-superset/issues/4374#issuecomment-365084856 I'd like to keep the larger files in the repo if not in the python package. Look at the `MANIFEST.in` to removed them from

[GitHub] mistercrunch commented on issue #4393: Bypassing the login for Supreset using iframe

2018-02-12 Thread GitBox
mistercrunch commented on issue #4393: Bypassing the login for Supreset using iframe URL: https://github.com/apache/incubator-superset/issues/4393#issuecomment-365087037 `pip/setuptools/python` manages your pypi packages, no need to worry about that. `superset_config.py` can live

[GitHub] mistercrunch commented on issue #2436: Slow query in SQL Lab

2018-02-12 Thread GitBox
mistercrunch commented on issue #2436: Slow query in SQL Lab URL: https://github.com/apache/incubator-superset/issues/2436#issuecomment-365090962 @THDA are sounds like you're looking at the Table component in the "explore view" (not in SQL Lab). For the "explore view" we should use

[GitHub] Tsuki commented on issue #1231: Support for emailing reports at regular intervals?

2018-02-12 Thread GitBox
Tsuki commented on issue #1231: Support for emailing reports at regular intervals? URL: https://github.com/apache/incubator-superset/issues/1231#issuecomment-365047325 +1 This is an automated message from the Apache Git

[GitHub] mistercrunch commented on issue #4405: Disable user access request

2018-02-12 Thread GitBox
mistercrunch commented on issue #4405: Disable user access request URL: https://github.com/apache/incubator-superset/pull/4405#issuecomment-365075267 You may want to disable the ModelView as well This is an automated message

[GitHub] goldymanikoth commented on issue #4393: Bypassing the login for Supreset using iframe

2018-02-12 Thread GitBox
goldymanikoth commented on issue #4393: Bypassing the login for Supreset using iframe URL: https://github.com/apache/incubator-superset/issues/4393#issuecomment-365143549 I got some reference to locate the superset install path and I figure out the location and updated the

[GitHub] mistercrunch commented on a change in pull request #4409: Add permissions decider for delegating access controls.

2018-02-12 Thread GitBox
mistercrunch commented on a change in pull request #4409: Add permissions decider for delegating access controls. URL: https://github.com/apache/incubator-superset/pull/4409#discussion_r167758341 ## File path: superset/viz.py ## @@ -307,7 +307,12 @@ def

[GitHub] mistercrunch commented on a change in pull request #4409: Add permissions decider for delegating access controls.

2018-02-12 Thread GitBox
mistercrunch commented on a change in pull request #4409: Add permissions decider for delegating access controls. URL: https://github.com/apache/incubator-superset/pull/4409#discussion_r167757524 ## File path: superset/config.py ## @@ -364,6 +364,24 @@ class

[GitHub] mistercrunch commented on issue #4393: Bypassing the login for Supreset using iframe

2018-02-12 Thread GitBox
mistercrunch commented on issue #4393: Bypassing the login for Supreset using iframe URL: https://github.com/apache/incubator-superset/issues/4393#issuecomment-365147723 You probably want to avoid changing `superset/config.py` (which means you're modifying the Superset package) and use

[GitHub] mistercrunch commented on issue #4405: Disable user access request

2018-02-12 Thread GitBox
mistercrunch commented on issue #4405: Disable user access request URL: https://github.com/apache/incubator-superset/pull/4405#issuecomment-365147909 Let's flip it off by default. This is an automated message from the Apache

[GitHub] timifasubaa opened a new pull request #4409: Add permissions decider for delegating access controls.

2018-02-12 Thread GitBox
timifasubaa opened a new pull request #4409: Add permissions decider for delegating access controls. URL: https://github.com/apache/incubator-superset/pull/4409 This PR creates the perms_decider object in the config which gives different deployments the ability to delegate data access

[GitHub] fabianmenges commented on issue #4409: Add permissions decider for delegating access controls.

2018-02-12 Thread GitBox
fabianmenges commented on issue #4409: Add permissions decider for delegating access controls. URL: https://github.com/apache/incubator-superset/pull/4409#issuecomment-365108884 @jeffreythewang @GeorgeSirois FYI This is an

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365165514 ## column configuration for font option: - set all cells to bold font by setting a wildcard conditional

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365160586 Some test cases ## column configuration: - Set all cell's background color to red (by selecting

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365168066 Hi, @mistercrunch Above is the overall features of this so-called "column and row configuration".

[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-02-12 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables URL: https://github.com/apache/incubator-superset/issues/3416#issuecomment-365168344 HI, all, we have updated lots of screenshots in https://github.com/apache/incubator-superset/pull/4394, please vote for

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365160586 Test cases for column configuration: - Set all cell's background color to red (by selecting only the

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365166059 ## Row configuration: - Set rows's background color to red if content in that row contains specific

[GitHub] Chun-LingChen opened a new pull request #4411: use full path in case of there are query params

2018-02-12 Thread GitBox
Chun-LingChen opened a new pull request #4411: use full path in case of there are query params URL: https://github.com/apache/incubator-superset/pull/4411 This is an automated message from the Apache Git Service. To respond

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365166554 ## Priority between column configuration and row configuration: - Column configuration has high priority

[GitHub] goldymanikoth commented on issue #4393: Bypassing the login for Supreset using iframe

2018-02-12 Thread GitBox
goldymanikoth commented on issue #4393: Bypassing the login for Supreset using iframe URL: https://github.com/apache/incubator-superset/issues/4393#issuecomment-365149514 got it, I will do that. Thanks This is an automated

[GitHub] mistercrunch closed pull request #4404: Fix ImportError when using BigQuery in SQL Editor

2018-02-12 Thread GitBox
mistercrunch closed pull request #4404: Fix ImportError when using BigQuery in SQL Editor URL: https://github.com/apache/incubator-superset/pull/4404 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mistercrunch commented on issue #4401: [SqlLab] Fix a few height related UI issues

2018-02-12 Thread GitBox
mistercrunch commented on issue #4401: [SqlLab] Fix a few height related UI issues URL: https://github.com/apache/incubator-superset/pull/4401#issuecomment-365153837 ? This is an automated message from the Apache Git

[GitHub] mistercrunch commented on issue #4384: Fix the bug of chord diagram shows incorrect flow from target to source

2018-02-12 Thread GitBox
mistercrunch commented on issue #4384: Fix the bug of chord diagram shows incorrect flow from target to source URL: https://github.com/apache/incubator-superset/pull/4384#issuecomment-365154074 I think it's ok to flip the chart metadata along with flipping the behavior. The chart should

[GitHub] mistercrunch commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
mistercrunch commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365154867 Oh neat! Can we get a few screenshots to get an idea of how this looks?

[GitHub] sanoopmm commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters

2018-02-12 Thread GitBox
sanoopmm commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters URL: https://github.com/apache/incubator-superset/issues/4366#issuecomment-365155863 I need to add filter drop downs in the existing visualizations like table view,

[GitHub] mistercrunch commented on issue #4397: Weird query generated when using Time Series Line Chart with Group By column

2018-02-12 Thread GitBox
mistercrunch commented on issue #4397: Weird query generated when using Time Series Line Chart with Group By column URL: https://github.com/apache/incubator-superset/issues/4397#issuecomment-365155670 It is the expected query if you want to get the time series for the top N dimension

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365156999 Here's a gif:

[GitHub] mistercrunch opened a new pull request #4410: Unset 'series limit' default from 50 to null

2018-02-12 Thread GitBox
mistercrunch opened a new pull request #4410: Unset 'series limit' default from 50 to null URL: https://github.com/apache/incubator-superset/pull/4410 Setting "Series Limit" by default results in a subquery (or 2 phase) which can be suboptimal / unecessary where low cardinality dims

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365156999 Here's a gif:

[GitHub] vnnw closed issue #4397: Weird query generated when using Time Series Line Chart with Group By column

2018-02-12 Thread GitBox
vnnw closed issue #4397: Weird query generated when using Time Series Line Chart with Group By column URL: https://github.com/apache/incubator-superset/issues/4397 This is an automated message from the Apache Git Service.

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365160586 Test cases for column configuration: - Set all cell's background color to red (by selecting only the

[GitHub] xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table

2018-02-12 Thread GitBox
xiaohanyu commented on issue #4394: Implement the column config and row config in pivot table URL: https://github.com/apache/incubator-superset/pull/4394#issuecomment-365160554 Hi, here's the "welcome" page for slice with pivot table viz type, we add two new sections in the left side