Re: Apache Superset Committer Meeting Notes

2018-03-12 Thread Nishant Bangarwa
He Jeff, I work in IST timezone and would love to be part of these meetings. Please add me. Also, is it possible to have the meeting little early as it will be too late for me. -- Nishant Bangarwa Hortonworks (M): +91-9729200044 On 3/12/18, 10:31 AM, "Jeff Feng"

[GitHub] geekSnails opened a new issue #4600: the size of slices in dashboard is too small

2018-03-12 Thread GitBox
geekSnails opened a new issue #4600: the size of slices in dashboard is too small URL: https://github.com/apache/incubator-superset/issues/4600 Make sure these boxes are checked before submitting your issue - thank you! - [ ] I have checked the superset logs for python stacktraces

[GitHub] mistercrunch commented on issue #4593: [sql lab] reduce the number of metadata calls when loading a table

2018-03-12 Thread GitBox
mistercrunch commented on issue #4593: [sql lab] reduce the number of metadata calls when loading a table URL: https://github.com/apache/incubator-superset/pull/4593#issuecomment-372406484 Let me add a unit test. This is an

[GitHub] xrmx commented on issue #4595: sqllab: preview should quote table name

2018-03-12 Thread GitBox
xrmx commented on issue #4595: sqllab: preview should quote table name URL: https://github.com/apache/incubator-superset/issues/4595#issuecomment-372406885 If someone can give me an hint at where the query is sent to the frontend i can give take a look at fixing it.

[GitHub] codecov-io commented on issue #4593: [sql lab] reduce the number of metadata calls when loading a table

2018-03-12 Thread GitBox
codecov-io commented on issue #4593: [sql lab] reduce the number of metadata calls when loading a table URL: https://github.com/apache/incubator-superset/pull/4593#issuecomment-372095936 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4593?src=pr=h1) Report > Merging

[GitHub] brylie commented on issue #4193: Init docker for local development environment.

2018-03-12 Thread GitBox
brylie commented on issue #4193: Init docker for local development environment. URL: https://github.com/apache/incubator-superset/pull/4193#issuecomment-372336673 @xiaohanyu, any status update here? Perhaps we can compromise and put these files in a `docker` folder, so it is

[DRAFT] Incubator PMC Board Report - March 2018

2018-03-12 Thread John D. Ament
All, Below is the current draft of the Incubator board report. We are still missing reports from many podlings, and several missing sign off. I have CC'd the podlings missing sign off, and separately notified the ones missing reports. John Incubator PMC report for March 2018 The Apache

[GitHub] xiaohanyu commented on issue #4193: Init docker for local development environment.

2018-03-12 Thread GitBox
xiaohanyu commented on issue #4193: Init docker for local development environment. URL: https://github.com/apache/incubator-superset/pull/4193#issuecomment-372337404 @brylie I'll update the status this week. Just come back to work from this week. Sorry for the delay. Think can put

Re: Apache Superset Committer Meeting Notes

2018-03-12 Thread Jim Jagielski
Is it possible to have rotating meetings? Although such meetings can be useful, they also have the disadvantage to disenfranchise large segments of the current and potential community, especially as the meetings become more popular and seen as "the" way of communicating and development. It is

Re: Apache Superset Committer Meeting Notes

2018-03-12 Thread Stephanie Rivera
Hi Jeff et al, I am interested in joining. We have begun to use Superset here at SpotX and my data engineering team is beginning to build out features for Superset. We do not have anything complete enough to commit yet. Mostly the team has been learning the framework and models as to determine

[GitHub] fabianmenges commented on a change in pull request #4593: [sql lab] reduce the number of metadata calls when loading a table

2018-03-12 Thread GitBox
fabianmenges commented on a change in pull request #4593: [sql lab] reduce the number of metadata calls when loading a table URL: https://github.com/apache/incubator-superset/pull/4593#discussion_r173848660 ## File path: superset/db_engine_specs.py ## @@ -235,14 +235,15

[GitHub] mistercrunch commented on a change in pull request #4593: [sql lab] reduce the number of metadata calls when loading a table

2018-03-12 Thread GitBox
mistercrunch commented on a change in pull request #4593: [sql lab] reduce the number of metadata calls when loading a table URL: https://github.com/apache/incubator-superset/pull/4593#discussion_r173887476 ## File path: superset/db_engine_specs.py ## @@ -235,14 +235,15

[GitHub] betodealmeida opened a new pull request #4601: Instantiate ChartContainer with width

2018-03-12 Thread GitBox
betodealmeida opened a new pull request #4601: Instantiate ChartContainer with width URL: https://github.com/apache/incubator-superset/pull/4601 Property `width` is not being passed properly when instantiating a `ChartContainer`. This results in an error when switching the viz type from

[GitHub] betodealmeida commented on issue #4601: Instantiate ChartContainer with width

2018-03-12 Thread GitBox
betodealmeida commented on issue #4601: Instantiate ChartContainer with width URL: https://github.com/apache/incubator-superset/pull/4601#issuecomment-372439650 This is broken, the width is too big. Will fix. This is an

[GitHub] betodealmeida closed pull request #4601: Instantiate ChartContainer with width

2018-03-12 Thread GitBox
betodealmeida closed pull request #4601: Instantiate ChartContainer with width URL: https://github.com/apache/incubator-superset/pull/4601 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] codecov-io commented on issue #4601: Instantiate ChartContainer with width

2018-03-12 Thread GitBox
codecov-io commented on issue #4601: Instantiate ChartContainer with width URL: https://github.com/apache/incubator-superset/pull/4601#issuecomment-372443272 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4601?src=pr=h1) Report > Merging

[GitHub] betodealmeida opened a new pull request #4601: Only show overlay if container is set

2018-03-12 Thread GitBox
betodealmeida opened a new pull request #4601: Only show overlay if container is set URL: https://github.com/apache/incubator-superset/pull/4601 Property `width` is not being passed properly when instantiating a `ChartContainer`. This results in an error when switching the viz type from

[GitHub] codecov-io commented on issue #4573: [cache] Ensuring that the datasource UID is defined

2018-03-12 Thread GitBox
codecov-io commented on issue #4573: [cache] Ensuring that the datasource UID is defined URL: https://github.com/apache/incubator-superset/pull/4573#issuecomment-371409484 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4573?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4573: [cache] Ensuring that the datasource UID is defined

2018-03-12 Thread GitBox
codecov-io commented on issue #4573: [cache] Ensuring that the datasource UID is defined URL: https://github.com/apache/incubator-superset/pull/4573#issuecomment-371409484 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4573?src=pr=h1) Report > Merging

[GitHub] jeffreythewang commented on issue #4520: Allow users to view dashboards they own

2018-03-12 Thread GitBox
jeffreythewang commented on issue #4520: Allow users to view dashboards they own URL: https://github.com/apache/incubator-superset/pull/4520#issuecomment-372383192 Any updates on this? This is an automated message from the

[GitHub] kkalyan commented on issue #4169: Dashboard Filtering Issue across different data sources

2018-03-12 Thread GitBox
kkalyan commented on issue #4169: Dashboard Filtering Issue across different data sources URL: https://github.com/apache/incubator-superset/issues/4169#issuecomment-372553335 @john-bodley @oxydash will be able to take a look at this issue?

[GitHub] john-bodley opened a new pull request #4602: [druid] Adding cluster filter for refresh

2018-03-12 Thread GitBox
john-bodley opened a new pull request #4602: [druid] Adding cluster filter for refresh URL: https://github.com/apache/incubator-superset/pull/4602 This PR fixes an issue with syncing of Druid datasources where the datasource name is not globally unique. Previously the filter didn't

[GitHub] codecov-io commented on issue #4579: Removing [dashboard] and [slice] titles to show name

2018-03-12 Thread GitBox
codecov-io commented on issue #4579: Removing [dashboard] and [slice] titles to show name URL: https://github.com/apache/incubator-superset/pull/4579#issuecomment-371642722 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4579?src=pr=h1) Report > Merging

[GitHub] williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text

2018-03-12 Thread GitBox
williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text URL: https://github.com/apache/incubator-superset/pull/4194#issuecomment-372541646 I think annotations are broken in production, so will debug that as part of this.

[GitHub] mistercrunch commented on issue #4604: Show "Range Filter" by default

2018-03-12 Thread GitBox
mistercrunch commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372542188 Personally I'm not a huge fan of this, I've seen users get confused by it and it looks pretty bad at reduced height (when added to a

[GitHub] mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot

2018-03-12 Thread GitBox
mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#discussion_r173973450 ## File path: superset/assets/visualizations/Legend.jsx ## @@ -0,0 +1,52 @@ +import React from

[GitHub] mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot

2018-03-12 Thread GitBox
mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#discussion_r174012429 ## File path: superset/assets/visualizations/deckgl/layers/scatter.jsx ## @@ -68,8 +89,10 @@

[GitHub] mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot

2018-03-12 Thread GitBox
mistercrunch commented on a change in pull request #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#discussion_r174012269 ## File path: superset/assets/visualizations/Legend.jsx ## @@ -0,0 +1,52 @@ +import React from

[GitHub] john-bodley commented on issue #4587: [logs] Dropping dt column

2018-03-12 Thread GitBox
john-bodley commented on issue #4587: [logs] Dropping dt column URL: https://github.com/apache/incubator-superset/pull/4587#issuecomment-372499774 @mistercrunch I added an `UPDATING.md` file. Let me know if you want me to add more refinement beyond a simple list of breaking PRs.

[GitHub] john-bodley closed issue #3541: Druid datasource names should only be unique for a given cluster

2018-03-12 Thread GitBox
john-bodley closed issue #3541: Druid datasource names should only be unique for a given cluster URL: https://github.com/apache/incubator-superset/issues/3541 This is an automated message from the Apache Git Service. To

[GitHub] john-bodley opened a new pull request #4603: [flask-appbuilder] Bumping version to 1.10.0

2018-03-12 Thread GitBox
john-bodley opened a new pull request #4603: [flask-appbuilder] Bumping version to 1.10.0 URL: https://github.com/apache/incubator-superset/pull/4603 This PR bumps the `Flask-AppBuilder` package to version

[GitHub] graceguo-supercat commented on issue #4597: [BugFix] Resizing widgets problem #4596

2018-03-12 Thread GitBox
graceguo-supercat commented on issue #4597: [BugFix] Resizing widgets problem #4596 URL: https://github.com/apache/incubator-superset/pull/4597#issuecomment-372468330 LGTM This is an automated message from the Apache Git

[GitHub] graceguo-supercat closed pull request #4597: [BugFix] Resizing widgets problem #4596

2018-03-12 Thread GitBox
graceguo-supercat closed pull request #4597: [BugFix] Resizing widgets problem #4596 URL: https://github.com/apache/incubator-superset/pull/4597 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 #4603: [flask-appbuilder] Bumping version to 1.10.0

2018-03-12 Thread GitBox
codecov-io commented on issue #4603: [flask-appbuilder] Bumping version to 1.10.0 URL: https://github.com/apache/incubator-superset/pull/4603#issuecomment-372525117 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4603?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4603: [flask-appbuilder] Bumping version to 1.10.0

2018-03-12 Thread GitBox
codecov-io commented on issue #4603: [flask-appbuilder] Bumping version to 1.10.0 URL: https://github.com/apache/incubator-superset/pull/4603#issuecomment-372525117 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4603?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4604: Show "Range Filter" by default

2018-03-12 Thread GitBox
codecov-io commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372526668 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4604?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4604: Show "Range Filter" by default

2018-03-12 Thread GitBox
codecov-io commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372526668 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4604?src=pr=h1) Report > Merging

[GitHub] betodealmeida opened a new pull request #4604: Show "Range Filter" by default

2018-03-12 Thread GitBox
betodealmeida opened a new pull request #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604 cc: @vylc This is an automated message from the Apache Git Service. To respond to the

[GitHub] williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text

2018-03-12 Thread GitBox
williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text URL: https://github.com/apache/incubator-superset/pull/4194#issuecomment-372494337 @mistercrunch will try to get this done today This

[GitHub] john-bodley commented on issue #4581: Making timeseries_limit not required for phase 2

2018-03-12 Thread GitBox
john-bodley commented on issue #4581: Making timeseries_limit not required for phase 2 URL: https://github.com/apache/incubator-superset/pull/4581#issuecomment-372485931 @mistercrunch would you mind reviewing this PR? It seems like you have the most context here.

[GitHub] codecov-io commented on issue #4602: [druid] Adding cluster filter for refresh

2018-03-12 Thread GitBox
codecov-io commented on issue #4602: [druid] Adding cluster filter for refresh URL: https://github.com/apache/incubator-superset/pull/4602#issuecomment-372498819 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4602?src=pr=h1) Report > Merging

[GitHub] MeenaJain commented on issue #738: Dual Axis Line chart

2018-03-12 Thread GitBox
MeenaJain commented on issue #738: Dual Axis Line chart URL: https://github.com/apache/incubator-superset/issues/738#issuecomment-372519464 How can i make line graph as an area chart? This is an automated message from the

[GitHub] betodealmeida commented on issue #4572: Legend for deck.gl scatterplot

2018-03-12 Thread GitBox
betodealmeida commented on issue #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#issuecomment-372479523 Once this is merged I'll add support in the other deck.gl viz types.

[GitHub] timifasubaa commented on issue #4520: Allow users to view dashboards they own

2018-03-12 Thread GitBox
timifasubaa commented on issue #4520: Allow users to view dashboards they own URL: https://github.com/apache/incubator-superset/pull/4520#issuecomment-372455213 I don't have merge privileges but this LGTM. This is an

[GitHub] codecov-io commented on issue #4601: Only show overlay if container is set

2018-03-12 Thread GitBox
codecov-io commented on issue #4601: Only show overlay if container is set URL: https://github.com/apache/incubator-superset/pull/4601#issuecomment-372443272 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4601?src=pr=h1) Report > Merging