IanWE commented on issue #4575: Remove Permission from View Error
URL:
https://github.com/apache/incubator-superset/issues/4575#issuecomment-372922177
@leehwmj-junta Thank you
This is an automated message from the Apache Git
IanWE closed issue #4575: Remove Permission from View Error
URL: https://github.com/apache/incubator-superset/issues/4575
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitH
yzfranklin opened a new issue #4612: In the “SQL Lab”, the “Results” section
shows “Pending” constantly
URL: https://github.com/apache/incubator-superset/issues/4612
Following the instructions of "superset document", I deployed the "superset"
to my Linux server. Everything seems ok.
Ho
leehwmj-junta commented on issue #4575: Remove Permission from View Error
URL:
https://github.com/apache/incubator-superset/issues/4575#issuecomment-372920872
I had got same logs on terminal.
My superset version is 0.22
So, to solve this problem, I had deleted ~/.superset/superset
sunyongmofang closed issue #4502: How to use two filter_box to control two
slice, and not to be interfered on the same dashboard?(divide and rule)
URL: https://github.com/apache/incubator-superset/issues/4502
This is an auto
NaukhaizAhmad opened a new issue #4611: [Slice] 'Visualization Type' should
display the drop down (listing all the visualization types) rather than
displaying an editable string field.
URL: https://github.com/apache/incubator-superset/issues/4611
### Superset version
0.22.1
mistercrunch commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372895890
Wouldn't be against them having their own tab, though that might be a bit
much
--
mistercrunch commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372895777
LGTM
This is an automated message from the Ap
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay
on deck_gl charts
URL:
https://github.com/apache/incubator-superset/pull/4610#issuecomment-372895371
Also means the overlay won't show up at all even when changing the data,
@betodealmeida any thoughts?
-
mistercrunch commented on a change in pull request #4604: Show "Range Filter"
by default
URL:
https://github.com/apache/incubator-superset/pull/4604#discussion_r174345221
##
File path: superset/assets/visualizations/nvd3_vis.js
##
@@ -158,9 +159,10 @@ function nvd3Vis(sli
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&el=h1)
Report
> Merging
[#4604](https://codecov.io/gh/apa
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&el=h1)
Report
> Merging
[#4604](https://codecov.io/gh/apa
betodealmeida commented on issue #4604: Show "Range Filter" by default
URL:
https://github.com/apache/incubator-superset/pull/4604#issuecomment-372876554
![brush](https://user-images.githubusercontent.com/1534870/37378923-6ec9dd40-26ee-11e8-8504-a8d8d10eb411.gif)
-
GabeLoins commented on issue #4610: [Explore] removing refresh chart overlay on
deck_gl charts
URL:
https://github.com/apache/incubator-superset/pull/4610#issuecomment-372874331
That was part of the confusion. We should still merge this PR, or the
overlay will appear incorrectly when deck
codecov-io commented on issue #4610: [Explore] removing refresh chart overlay
on deck_gl charts
URL:
https://github.com/apache/incubator-superset/pull/4610#issuecomment-372872875
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4610?src=pr&el=h1)
Report
> Merging
[#46
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay
on deck_gl charts
URL:
https://github.com/apache/incubator-superset/pull/4610#issuecomment-372869409
After discussion, there was confusion around the fact that `autozoom` was
overwriting the `viewport` setting
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay
on deck_gl charts
URL:
https://github.com/apache/incubator-superset/pull/4610#issuecomment-372869409
After discussion, there was confusion around the fact that `autozoom` was
overwriting the `viewport` setting
GabeLoins opened a new pull request #4610: [Explore] removing refresh chart
overlay on deck_gl charts
URL: https://github.com/apache/incubator-superset/pull/4610
Deck gl charts have this weird thing where scrolling the canvas isn't
considered `renderTrigger: true` but also immediately upda
betodealmeida commented on a change in pull request #4572: Legend for deck.gl
scatterplot
URL:
https://github.com/apache/incubator-superset/pull/4572#discussion_r174322258
##
File path: superset/assets/visualizations/deckgl/layers/scatter.jsx
##
@@ -68,8 +89,10 @@ functio
betodealmeida commented on a change in pull request #4572: Legend for deck.gl
scatterplot
URL:
https://github.com/apache/incubator-superset/pull/4572#discussion_r174321354
##
File path: superset/assets/visualizations/Legend.jsx
##
@@ -0,0 +1,52 @@
+import React from 'reac
codecov-io commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372839264
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4609?src=pr&el=h1)
Report
> Merging
[#4609
codecov-io commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372839264
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4609?src=pr&el=h1)
Report
> Merging
[#4609
williaster commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901
thanks for updating this 🙌 LGTM, only possible suggestion would be making
`data` a variable constant instead for
williaster commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901
thanks for updating this 🙌 LGTM, only possible suggestion would be making
`data` a variable constant instead for
williaster commented on issue #4609: [Explore] overriding Annotations and
Layers' control tab
URL:
https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901
thanks for updating this 🙌 LGTM, only possible suggestion would be making
`data` a variable constant instead for
williaster closed pull request #4607: [slice_json] pass slice id to
get_form_data()
URL: https://github.com/apache/incubator-superset/pull/4607
This is an automated message from the Apache Git Service.
To respond to the mess
graceguo-supercat commented on issue #4607: [slice_json] pass slice id to
get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#issuecomment-372830359
LGTM!
This is an automated message from the Apache
graceguo-supercat commented on a change in pull request #4607: [slice_json]
pass slice id to get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#discussion_r174284347
##
File path: superset/views/core.py
##
@@ -1122,10 +1123,10 @@ def generate_json
GabeLoins opened a new pull request #4609: [Explore] overriding Annotations and
Layers' control tab
URL: https://github.com/apache/incubator-superset/pull/4609
as reported by @williaster- the annotations and layers control should live
in the Data tab and not the Style tab, despite its rend
graceguo-supercat commented on a change in pull request #4607: [slice_json]
pass slice id to get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#discussion_r174284347
##
File path: superset/views/core.py
##
@@ -1122,10 +1123,10 @@ def generate_json
jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53
- 1, longs are bigger
URL:
https://github.com/apache/incubator-superset/pull/4005#issuecomment-372789596
Any updates on this besides a rebase (and a possible test fix)?
-
jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53
- 1, longs are bigger
URL:
https://github.com/apache/incubator-superset/pull/4005#issuecomment-372789596
Any updates on this besides a rebase?
Th
fabianmenges commented on a change in pull request #4607: [slice_json] pass
slice id to get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#discussion_r174256060
##
File path: superset/views/core.py
##
@@ -1122,10 +1123,10 @@ def generate_json(self
john-bodley closed pull request #4590: Removing escape_sql so we dont double
escape
URL: https://github.com/apache/incubator-superset/pull/4590
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 t
williaster commented on a change in pull request #4607: [slice_json] pass slice
id to get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#discussion_r174253852
##
File path: superset/views/core.py
##
@@ -1122,10 +1123,10 @@ def generate_json(self,
codecov-io commented on issue #4608: Set filter_select_enabled default to True
for Druid
URL:
https://github.com/apache/incubator-superset/pull/4608#issuecomment-372785763
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4608?src=pr&el=h1)
Report
> Merging
[#4608](htt
codecov-io commented on issue #4608: Set filter_select_enabled default to True
for Druid
URL:
https://github.com/apache/incubator-superset/pull/4608#issuecomment-372785763
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4608?src=pr&el=h1)
Report
> Merging
[#4608](htt
fabianmenges commented on a change in pull request #4607: [slice_json] pass
slice id to get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#discussion_r174252687
##
File path: superset/views/core.py
##
@@ -1122,10 +1123,10 @@ def generate_json(self
codecov-io commented on issue #4194: [bug fixes] annotations <> x domains,
zeros in text
URL:
https://github.com/apache/incubator-superset/pull/4194#issuecomment-372785385
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4194?src=pr&el=h1)
Report
> Merging
[#4194](htt
codecov-io commented on issue #4607: [slice_json] pass slice id to
get_form_data()
URL:
https://github.com/apache/incubator-superset/pull/4607#issuecomment-372784539
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4607?src=pr&el=h1)
Report
> Merging
[#4607](https://c
john-bodley closed pull request #4579: Removing [dashboard] and [slice] titles
to show name
URL: https://github.com/apache/incubator-superset/pull/4579
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 provenanc
mistercrunch opened a new pull request #4608: Set filter_select_enabled default
to True for Druid
URL: https://github.com/apache/incubator-superset/pull/4608
This is an automated message from the Apache Git Service.
To respo
williaster commented on issue #4194: [bug fixes] annotations <> x domains,
zeros in text
URL:
https://github.com/apache/incubator-superset/pull/4194#issuecomment-372780297
fixed annotations separately in #4607, updated this PR / should be good to
go so PTAL ?
--
williaster opened a new pull request #4607: [slice_json] pass slice id to
get_form_data()
URL: https://github.com/apache/incubator-superset/pull/4607
@mistercrunch @graceguo-supercat @fabianmenges
[This
line](https://github.com/apache/incubator-superset/commit/83524f97d7fcf3c8c86ef
codecov-io commented on issue #4606: Use 'count' as the default metric when
available
URL:
https://github.com/apache/incubator-superset/pull/4606#issuecomment-372776199
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4606?src=pr&el=h1)
Report
> Merging
[#4606](https:
john-bodley commented on issue #4590: Removing escape_sql so we dont double
escape
URL:
https://github.com/apache/incubator-superset/pull/4590#issuecomment-372774659
@michellethomas upon further investigation this regression doesn't seem to
be related to the auto-escape feature but rather
john-bodley commented on issue #4590: Removing escape_sql so we dont double
escape
URL:
https://github.com/apache/incubator-superset/pull/4590#issuecomment-372774659
For context this SQLAlchemy
[commit](https://github.com/zzzeek/sqlalchemy/commit/2392ae1900f112c44ed966783d1dedfb88f13353)
codecov-io commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1)
Report
> Merging
[#4605](https://codecov.io/gh/apache/incu
john-bodley closed pull request #4573: [cache] Ensuring that the datasource UID
is defined
URL: https://github.com/apache/incubator-superset/pull/4573
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
michellethomas commented on issue #4573: [cache] Ensuring that the datasource
UID is defined
URL:
https://github.com/apache/incubator-superset/pull/4573#issuecomment-372759162
lgtm
This is an automated message from the Apach
john-bodley closed pull request #4602: [druid] Adding cluster filter for refresh
URL: https://github.com/apache/incubator-superset/pull/4602
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
michellethomas commented on issue #4602: [druid] Adding cluster filter for
refresh
URL:
https://github.com/apache/incubator-superset/pull/4602#issuecomment-372758640
lgtm
This is an automated message from the Apache Git Serv
mistercrunch opened a new pull request #4606: Use 'count' as the default metric
when available
URL: https://github.com/apache/incubator-superset/pull/4606
Count is a much better default than the current behavior which is to use
whatever the first metric in the list happens to be.
john-bodley commented on issue #4169: Dashboard Filtering Issue across
different data sources
URL:
https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566
@kkalyan I believe this change is a side effect of the caching restructure
(cc @mistercrunch) however one coul
john-bodley commented on issue #4169: Dashboard Filtering Issue across
different data sources
URL:
https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566
@kkalyan I believe this change is a side effect of the caching restructure
(cc @mistercrunch) however one coul
john-bodley commented on issue #4169: Dashboard Filtering Issue across
different data sources
URL:
https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566
@kkalayn I believe this change is a side effect of the caching restructure
(cc @mistercrunch) however one coul
rumbin commented on issue #2814: [explore] Filter Select not working for int
and bool
URL:
https://github.com/apache/incubator-superset/issues/2814#issuecomment-372742376
@ZamoraAndres check out #3007 please and whether the Boolean representation
in Impala is false/true (and not 0/1 like
rumbin commented on issue #2814: [explore] Filter Select not working for int
and bool
URL:
https://github.com/apache/incubator-superset/issues/2814#issuecomment-372739842
I guess, we need to file a new issue for the bool-specific part, as this one
is closed already.
@ZamoraAndres could
ZamoraAndres commented on issue #2814: [explore] Filter Select not working for
int and bool
URL:
https://github.com/apache/incubator-superset/issues/2814#issuecomment-372731509
Hello, I experience the same behavior connecting the tool to an Impala table.
The only workaround that I found
ZamoraAndres commented on issue #2814: [explore] Filter Select not working for
int and bool
URL:
https://github.com/apache/incubator-superset/issues/2814#issuecomment-372731509
Hello, I experience the same behavior connecting the tool to an Impala table.
The only workaround that I Found
codecov-io commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1)
Report
> Merging
[#4605](https://codecov.io/gh/apache/incu
codecov-io commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1)
Report
> Merging
[#4605](https://codecov.io/gh/apache/incu
mistercrunch closed pull request #4581: Making timeseries_limit not required
for phase 2
URL: https://github.com/apache/incubator-superset/pull/4581
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:
mistercrunch closed issue #4208: Druid run_query executes phase 1 query with no
limit
URL: https://github.com/apache/incubator-superset/issues/4208
This is an automated message from the Apache Git Service.
To respond to the
codecov-io commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014
#
[Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1)
Report
> Merging
[#4605](https://codecov.io/gh/apache/incu
xrmx commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372719111
You added the messages.json to pt_br and please rename the ptpt dir to pt
with git mv
T
maver1ck commented on issue #4543: Setting up compression using flask-compress
URL:
https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921
@mistercrunch
How to disable this behaviour?
I have nginx in front of Superset for ssl and compression.
For sure it
maver1ck commented on issue #4543: Setting up compression using flask-compress
URL:
https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921
@mistercrunch
How to disable this option ?
I have nginx in front of Superset for ssl and compression.
For sure it s
maver1ck commented on issue #4543: Setting up compression using flask-compress
URL:
https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921
How to disable this option ?
I have nginx in front of Superset for ssl and compression.
For sure it should be configura
maver1ck commented on issue #4543: Setting up compression using flask-compress
URL:
https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921
How to disable this option ?
I have nginx in front of Superset for ssl and compression.
For sure it should be configura
maver1ck commented on issue #4543: Setting up compression using flask-compress
URL:
https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921
How to disable this option ?
I have nginx in front of Superset for ssl and compression.
-
nunohelibeires commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372708237
@xrmx honestly it should be just pt but as I noticed there was a ptbr
version, I changed to ptpt.
added the missing json file
-
xrmx commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372654679
Are you sure portuguese is ptpt and not just pt or pt_PT?
This is an automated message f
xrmx commented on issue #4605: Portuguese Translation
URL:
https://github.com/apache/incubator-superset/pull/4605#issuecomment-372654822
Also you are missing the messages.json for the frontend
This is an automated message fro
maver1ck commented on issue #4428: Proper error handling in Hive Queries
URL:
https://github.com/apache/incubator-superset/pull/4428#issuecomment-372642657
ping @mistercrunch
This is an automated message from the Apache Git
nunohelibeires opened a new pull request #4605: Portuguese Translation
URL: https://github.com/apache/incubator-superset/pull/4605
complete.
my contribution to superset, with the help of @jcoelho-pt
This is an autom
scott-retair commented on issue #3613: Time Series Table: group-by keys must be
strings
URL:
https://github.com/apache/incubator-superset/issues/3613#issuecomment-372617052
I met the same issue
This is an automated message f
77 matches
Mail list logo