[GitHub] varaga opened a new issue #4794: Superset evaluation - SQLLite failure to refresh metadata

2018-04-10 Thread GitBox
varaga opened a new issue #4794: Superset evaluation - SQLLite failure to refresh metadata URL: https://github.com/apache/incubator-superset/issues/4794 Make sure these boxes are checked before submitting your issue - thank you! - [Y ] I have checked the superset logs for python

[GitHub] victorgil88 commented on issue #4691: superset db upgrade fails

2018-04-10 Thread GitBox
victorgil88 commented on issue #4691: superset db upgrade fails URL: https://github.com/apache/incubator-superset/issues/4691#issuecomment-380049700 At the end, this worked for me : `pip install superset` `cd \venv\Scripts` `python superset db upgrade` and the rest is

[GitHub] mistercrunch commented on issue #4755: Pass granularity from backend to frontend as ISO duration

2018-04-10 Thread GitBox
mistercrunch commented on issue #4755: Pass granularity from backend to frontend as ISO duration URL: https://github.com/apache/incubator-superset/pull/4755#issuecomment-380140800 @betodealmeida ^ This is an automated

[GitHub] mistercrunch commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG'

2018-04-10 Thread GitBox
mistercrunch commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG' URL: https://github.com/apache/incubator-superset/issues/4785#issuecomment-380143372 I tried with your superset_config and I still see logs. Anything

[GitHub] mistercrunch commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG'

2018-04-10 Thread GitBox
mistercrunch commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG' URL: https://github.com/apache/incubator-superset/issues/4785#issuecomment-380143372 I tried with your superset_config and I still see log. Anything

[GitHub] mistercrunch commented on issue #4794: Superset evaluation - SQLLite failure to refresh metadata

2018-04-10 Thread GitBox
mistercrunch commented on issue #4794: Superset evaluation - SQLLite failure to refresh metadata URL: https://github.com/apache/incubator-superset/issues/4794#issuecomment-380139798 I'd recommend to use MySQL or Postgres instead. You may want to try a single worker/thread gunicorn

[GitHub] mistercrunch commented on issue #4784: Gallery USES documentation.

2018-04-10 Thread GitBox
mistercrunch commented on issue #4784: Gallery USES documentation. URL: https://github.com/apache/incubator-superset/issues/4784#issuecomment-380140432 Some of this could be generated given the `visTypes.js` configuration along with `controls.js` (has labels and long descriptions for each

[GitHub] codecov-io commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
codecov-io commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380185912 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4795?src=pr=h1) Report > Merging

[GitHub] michellethomas opened a new issue #4796: title not shown on favorited charts on user profile

2018-04-10 Thread GitBox
michellethomas opened a new issue #4796: title not shown on favorited charts on user profile URL: https://github.com/apache/incubator-superset/issues/4796 On the profile page (/superset/profile/michelle_thomas/) favorited charts show the creator and the date it was favorited, but does not

[GitHub] mistercrunch opened a new pull request #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795 I set up a new environment today, while I kept my `superset_config.py` that happen to contain `os.environ['SUPERSET_UPDATE_PERMS'] = '0'`.

[GitHub] hughhhh opened a new pull request #4797: Deck.gl.test

2018-04-10 Thread GitBox
hug opened a new pull request #4797: Deck.gl.test URL: https://github.com/apache/incubator-superset/pull/4797 Added utils.py for having tooling for better testing. Put `load_fixtures()` method to load json into dict. Reference #4688

[GitHub] hughhhh opened a new pull request #4798: [DeckGL] Added fixtures and Deck test

2018-04-10 Thread GitBox
hug opened a new pull request #4798: [DeckGL] Added fixtures and Deck test URL: https://github.com/apache/incubator-superset/pull/4798 Add utils.py for better testing examples moving that can be loaded from a json @mistercrunch

[GitHub] john-bodley commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
john-bodley commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380202121 @mistercrunch any reason you wouldn't have `SUPERSET_UPDATE_PERMS` = `True` in this case?

[GitHub] john-bodley commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
john-bodley commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380211861 [Here](https://github.com/apache/incubator-superset/blob/64459efebda88cc147eacb14308d2b3e587899ef/superset/utils.py#L773) is

[GitHub] mistercrunch commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380208168 That would work too if you mean `os.environ['SUPERSET_UPDATE_PERMS'] = '1'`

[GitHub] mistercrunch commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380208168 That would work too if you mean `os.envrion['SUPERSET_UPDATE_PERMS'] = '1'`

[GitHub] hughhhh closed pull request #4688: Added DeckGL Test

2018-04-10 Thread GitBox
hug closed pull request #4688: Added DeckGL Test URL: https://github.com/apache/incubator-superset/pull/4688 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 provenance: As this is a foreign pull request

[GitHub] hughhhh closed pull request #4797: Deck.gl.test

2018-04-10 Thread GitBox
hug closed pull request #4797: Deck.gl.test URL: https://github.com/apache/incubator-superset/pull/4797 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 provenance: As this is a foreign pull request (from

[GitHub] mistercrunch opened a new pull request #4799: [bugfix] when num_period_compare is not set

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4799: [bugfix] when num_period_compare is not set URL: https://github.com/apache/incubator-superset/pull/4799 Fixing issues where y_axis_format is set and not num_period_compare This

[GitHub] codecov-io commented on issue #4801: [bugfix] dedup groupby columns in Deck visualizations

2018-04-10 Thread GitBox
codecov-io commented on issue #4801: [bugfix] dedup groupby columns in Deck visualizations URL: https://github.com/apache/incubator-superset/pull/4801#issuecomment-380263155 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4801?src=pr=h1) Report > Merging

[GitHub] michellethomas commented on issue #4587: [logs] Dropping dt column

2018-04-10 Thread GitBox
michellethomas commented on issue #4587: [logs] Dropping dt column URL: https://github.com/apache/incubator-superset/pull/4587#issuecomment-380279052 lgtm This is an automated message from the Apache Git Service. To respond

[GitHub] john-bodley closed pull request #4552: [travis/tox] Restructuring configuration and testing

2018-04-10 Thread GitBox
john-bodley closed pull request #4552: [travis/tox] Restructuring configuration and testing URL: https://github.com/apache/incubator-superset/pull/4552 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] graceguo-supercat opened a new pull request #4802: [homepage] Fix Favorites chart list

2018-04-10 Thread GitBox
graceguo-supercat opened a new pull request #4802: [homepage] Fix Favorites chart list URL: https://github.com/apache/incubator-superset/pull/4802 ![image](https://user-images.githubusercontent.com/27990562/38588749-606c7678-3cdc-11e8-832f-f56bb3aff873.png)

[GitHub] codecov-io commented on issue #4799: [bugfix] when num_period_compare is not set

2018-04-10 Thread GitBox
codecov-io commented on issue #4799: [bugfix] when num_period_compare is not set URL: https://github.com/apache/incubator-superset/pull/4799#issuecomment-380256020 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4799?src=pr=h1) Report > Merging

[GitHub] mistercrunch closed issue #4751: [P00 - Dashboard] Chart whitespace customization

2018-04-10 Thread GitBox
mistercrunch closed issue #4751: [P00 - Dashboard] Chart whitespace customization URL: https://github.com/apache/incubator-superset/issues/4751 This is an automated message from the Apache Git Service. To respond to the

[GitHub] graceguo-supercat commented on issue #4756: Improve xAxis ticks, thinner bottom margin

2018-04-10 Thread GitBox
graceguo-supercat commented on issue #4756: Improve xAxis ticks, thinner bottom margin URL: https://github.com/apache/incubator-superset/pull/4756#issuecomment-380235851 @mistercrunch This PR works good in airbnb data! please merge whenever you feel comfortable. Thanks!

[GitHub] mistercrunch opened a new pull request #4800: Improve xAxis ticks, thinner bottom margin (#4756)

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4800: Improve xAxis ticks, thinner bottom margin (#4756) URL: https://github.com/apache/incubator-superset/pull/4800 * Improve xAxis ticks, thinner bottom margin * Moving utils folder

[GitHub] mistercrunch commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380251830 @john-bodley the code I added here is in that same method. What this does is if running `superset init`, that method always

[GitHub] mistercrunch closed pull request #4799: [bugfix] when num_period_compare is not set

2018-04-10 Thread GitBox
mistercrunch closed pull request #4799: [bugfix] when num_period_compare is not set URL: https://github.com/apache/incubator-superset/pull/4799 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 provenance: As

[GitHub] mistercrunch opened a new pull request #4801: [bugfix] dedup groupby columns in Deck visualizations

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4801: [bugfix] dedup groupby columns in Deck visualizations URL: https://github.com/apache/incubator-superset/pull/4801 When specifying the same column twice as a `Extra Data for JS` and `Categorical Color`, an error is issued. This addresses this

[GitHub] zhangshuge commented on issue #4784: Gallery USES documentation.

2018-04-10 Thread GitBox
zhangshuge commented on issue #4784: Gallery USES documentation. URL: https://github.com/apache/incubator-superset/issues/4784#issuecomment-380021634 @xrmx @mistercrunchyes! I want a detailed document on the visual configuration of a graphic, detailing each of the specific

[GitHub] mistercrunch commented on issue #4771: [druid] fix 'Unorderable types' when col has nulls

2018-04-10 Thread GitBox
mistercrunch commented on issue #4771: [druid] fix 'Unorderable types' when col has nulls URL: https://github.com/apache/incubator-superset/pull/4771#issuecomment-380324427 Merging onto https://github.com/apache/incubator-superset/pull/4651

[GitHub] mistercrunch closed pull request #4771: [druid] fix 'Unorderable types' when col has nulls

2018-04-10 Thread GitBox
mistercrunch closed pull request #4771: [druid] fix 'Unorderable types' when col has nulls URL: https://github.com/apache/incubator-superset/pull/4771 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] emirot commented on issue #4780: [i18n] Add French translation

2018-04-10 Thread GitBox
emirot commented on issue #4780: [i18n] Add French translation URL: https://github.com/apache/incubator-superset/pull/4780#issuecomment-380327229 @xrmx I've created https://github.com/dpgaspar/Flask-AppBuilder/pull/735 This

[GitHub] lrcutra opened a new issue #4805: "Custom" dimensions in Druid datasources cause problems with some Visualization types

2018-04-10 Thread GitBox
lrcutra opened a new issue #4805: "Custom" dimensions in Druid datasources cause problems with some Visualization types URL: https://github.com/apache/incubator-superset/issues/4805 Make sure these boxes are checked before submitting your issue - thank you! - [x] I have checked the

[GitHub] deepak-goel commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG'

2018-04-10 Thread GitBox
deepak-goel commented on issue #4785: Not able to get the debug logs or python stacktrace on errors even after LOG_LEVEL is set to 'DEBUG' URL: https://github.com/apache/incubator-superset/issues/4785#issuecomment-37284 Nothing special about the setup. There is a standard installation

[GitHub] xrmx commented on issue #4784: Gallery USES documentation.

2018-04-10 Thread GitBox
xrmx commented on issue #4784: Gallery USES documentation. URL: https://github.com/apache/incubator-superset/issues/4784#issuecomment-380006678 AFAIU they are asking for documentation on how to configure each visualizations... quickly.

[GitHub] xrmx commented on issue #4691: superset db upgrade fails

2018-04-10 Thread GitBox
xrmx commented on issue #4691: superset db upgrade fails URL: https://github.com/apache/incubator-superset/issues/4691#issuecomment-380007652 @victorgil88 then it's not the same problem, you probably have installed superset in a virtualenv that you forgot to activate

[GitHub] xrmx commented on issue #4691: superset db upgrade fails

2018-04-10 Thread GitBox
xrmx commented on issue #4691: superset db upgrade fails URL: https://github.com/apache/incubator-superset/issues/4691#issuecomment-380007652 @victorgil88 then it's not the same problem, you probably have installed in a virtualenv that you forgot to activate

[GitHub] xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration

2018-04-10 Thread GitBox
xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration URL: https://github.com/apache/incubator-superset/pull/4755#issuecomment-380008827 @NathanQuinn i think the ".function" part should be removed. And we are probably missing some tests :)

[GitHub] xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration

2018-04-10 Thread GitBox
xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration URL: https://github.com/apache/incubator-superset/pull/4755#issuecomment-380008827 @NathanQuinn i think the ".function" part should be removed

[GitHub] xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration

2018-04-10 Thread GitBox
xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration URL: https://github.com/apache/incubator-superset/pull/4755#issuecomment-380008827 @NathanQuinn ~~i think the ".function" part should be removed~~~. It's the default of the get that is wrong, it

[GitHub] xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration

2018-04-10 Thread GitBox
xrmx commented on issue #4755: Pass granularity from backend to frontend as ISO duration URL: https://github.com/apache/incubator-superset/pull/4755#issuecomment-380008827 @NathanQuinn ~~i think the ".function" part should be removed~~~. It's the default of the get that is wrong. And we

[GitHub] codecov-io commented on issue #4793: [line] do not show 0 or nulls in rich tooltip

2018-04-10 Thread GitBox
codecov-io commented on issue #4793: [line] do not show 0 or nulls in rich tooltip URL: https://github.com/apache/incubator-superset/pull/4793#issuecomment-379988834 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4793?src=pr=h1) Report > Merging

[GitHub] victorgil88 commented on issue #4691: superset db upgrade fails

2018-04-10 Thread GitBox
victorgil88 commented on issue #4691: superset db upgrade fails URL: https://github.com/apache/incubator-superset/issues/4691#issuecomment-379997371 I have the same problem. In my case, the answer is: "superset" is not recognized as a command

[GitHub] mistercrunch opened a new pull request #4793: [line] do not show 0 or nulls in rich tooltip

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4793: [line] do not show 0 or nulls in rich tooltip URL: https://github.com/apache/incubator-superset/pull/4793 https://user-images.githubusercontent.com/487433/38539031-364a9fe6-3c4b-11e8-93a4-ff17463fdcc8.png;>

[GitHub] john-bodley closed pull request #4587: [logs] Dropping dt column

2018-04-10 Thread GitBox
john-bodley closed pull request #4587: [logs] Dropping dt column URL: https://github.com/apache/incubator-superset/pull/4587 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 provenance: As this is a foreign

[GitHub] john-bodley commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
john-bodley commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380302796 Agreed, I guess what I'm saying is I'm not certain whether we want to always ensure that `superset init` overrides the logic

[GitHub] mistercrunch commented on issue #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch commented on issue #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795#issuecomment-380314794 I can change our deploy script instead This is an automated

[GitHub] mistercrunch closed pull request #4795: [cli] force update perms on superset init

2018-04-10 Thread GitBox
mistercrunch closed pull request #4795: [cli] force update perms on superset init URL: https://github.com/apache/incubator-superset/pull/4795 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 provenance: As

[GitHub] codecov-io commented on issue #4800: Improve the calendar heatmap

2018-04-10 Thread GitBox
codecov-io commented on issue #4800: Improve the calendar heatmap URL: https://github.com/apache/incubator-superset/pull/4800#issuecomment-380320073 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4800?src=pr=h1) Report > Merging

[GitHub] mistercrunch closed pull request #4525: Superset issue #4512: multiple metrics, group by, opacity, legends for histogram

2018-04-10 Thread GitBox
mistercrunch closed pull request #4525: Superset issue #4512: multiple metrics, group by, opacity, legends for histogram URL: https://github.com/apache/incubator-superset/pull/4525 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] mistercrunch commented on issue #4525: Superset issue #4512: multiple metrics, group by, opacity, legends for histogram

2018-04-10 Thread GitBox
mistercrunch commented on issue #4525: Superset issue #4512: multiple metrics, group by, opacity, legends for histogram URL: https://github.com/apache/incubator-superset/pull/4525#issuecomment-380322503 @ArielStv, I finally took the time to pull the branch and play with this. This is

[GitHub] codecov-io commented on issue #4802: [homepage] Fix Favorites chart list

2018-04-10 Thread GitBox
codecov-io commented on issue #4802: [homepage] Fix Favorites chart list URL: https://github.com/apache/incubator-superset/pull/4802#issuecomment-380282489 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4802?src=pr=h1) Report > Merging

[GitHub] michellethomas commented on issue #4802: [homepage] Fix Favorites chart list

2018-04-10 Thread GitBox
michellethomas commented on issue #4802: [homepage] Fix Favorites chart list URL: https://github.com/apache/incubator-superset/pull/4802#issuecomment-380284056 fixes: #4796 This is an automated message from the Apache Git

[GitHub] graceguo-supercat closed pull request #4802: [homepage] Fix Favorites chart list

2018-04-10 Thread GitBox
graceguo-supercat closed pull request #4802: [homepage] Fix Favorites chart list URL: https://github.com/apache/incubator-superset/pull/4802 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 provenance: As this

[GitHub] mistercrunch opened a new pull request #4803: [explore] set working default for MetricsControl

2018-04-10 Thread GitBox
mistercrunch opened a new pull request #4803: [explore] set working default for MetricsControl URL: https://github.com/apache/incubator-superset/pull/4803 The default setting which would look for either `count` first and then any metric stopped working when we landed MetricsControl.

[GitHub] codecov-io commented on issue #4804: [upload csv to hive] fix call to unicodereader.next()

2018-04-10 Thread GitBox
codecov-io commented on issue #4804: [upload csv to hive] fix call to unicodereader.next() URL: https://github.com/apache/incubator-superset/pull/4804#issuecomment-380304062 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4804?src=pr=h1) Report > Merging

[GitHub] john-bodley commented on issue #4804: [upload csv to hive] fix call to unicodereader.next()

2018-04-10 Thread GitBox
john-bodley commented on issue #4804: [upload csv to hive] fix call to unicodereader.next() URL: https://github.com/apache/incubator-superset/pull/4804#issuecomment-380304142 @timifasubaa it seems in Python 2 uses

[GitHub] michellethomas commented on issue #4802: [homepage] Fix Favorites chart list

2018-04-10 Thread GitBox
michellethomas commented on issue #4802: [homepage] Fix Favorites chart list URL: https://github.com/apache/incubator-superset/pull/4802#issuecomment-380284056 fixes #4796 This is an automated message from the Apache Git

[GitHub] michellethomas closed issue #4796: title not shown on favorited charts on user profile

2018-04-10 Thread GitBox
michellethomas closed issue #4796: title not shown on favorited charts on user profile URL: https://github.com/apache/incubator-superset/issues/4796 This is an automated message from the Apache Git Service. To respond to

[GitHub] codecov-io commented on issue #4803: [explore] set working default for MetricsControl

2018-04-10 Thread GitBox
codecov-io commented on issue #4803: [explore] set working default for MetricsControl URL: https://github.com/apache/incubator-superset/pull/4803#issuecomment-380298865 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4803?src=pr=h1) Report > Merging

[GitHub] timifasubaa opened a new pull request #4804: [upload csv to hive] fix call to unicodereader.next()

2018-04-10 Thread GitBox
timifasubaa opened a new pull request #4804: [upload csv to hive] fix call to unicodereader.next() URL: https://github.com/apache/incubator-superset/pull/4804 Calling unicodeReader.next() was problematic because i doesn't reliably work. The correct way to get the first item in the list is