You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "maudrid (via GitHub)" <gi...@apache.org> on 2023/06/12 13:27:07 UTC
[GitHub] [superset] maudrid opened a new issue, #24357: Predictive analytics not working out of the box when using docker.
maudrid opened a new issue, #24357:
URL: https://github.com/apache/superset/issues/24357
When deploying Superset using the official docker image, the predictive analytics functionality does not work.
This is due to missing dependencies.
#### How to reproduce the bug
1. Run Supserset using docker and the `apache/superset:2.1.0` image.
2. Create any Timeseries chart and check the "ENABLE FORECAST' box under 'Predictive analytics'.
3. Update the chart and receive an error message: 'Error loading data from cache'.
### Expected results
The chart to show predictions.
### Actual results
Update the chart and receive an error message: 'Error loading data from cache' and the log contains:
```
force_cached (QueryContext): value not found for key f1510b265fdd1416331715b54d95d017
WARNING:superset.common.utils.query_cache_manager:force_cached (QueryContext): value not found for key f1510b265fdd1416331715b54d95d017
```
### Workaround / Fix
In order to remove the error, you need to build a new docker image with the following lines:
```
FROM apache/superset:2.1.0
USER root
RUN pip install --upgrade pip
RUN pip install lunarcalendar tqdm "pystan<3.0" && pip install "prophet>=1.0.1, <1.1"
```
Can these missing dependencies not be included in the docker image, so that the functionality will work out of the box?
### Checklist
Make sure to follow these steps before submitting your issue - thank you!
- [ x ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
- [ x ] I have reproduced the issue with at least the latest released version of superset.
- [ x ] I have checked the issue tracker for the same issue and I haven't found one similar. There are several issues where people struggle to get this to work, why not include it in the image so that we don't have to manually add this?
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1969981579
@Razikus what version of Superset are you running? The rest of the thread is from the 2.x era. I wouldn't think we're seeing this in 3.1, and we should close this issue if not.
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "maZahaca (via GitHub)" <gi...@apache.org>.
maZahaca commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1792715275
Any ready to go docker images available? like the one you have for `-dev` dev?
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] sebastianliebscher commented on issue #24357: Predictive analytics not working out of the box when using docker.
Posted by "sebastianliebscher (via GitHub)" <gi...@apache.org>.
sebastianliebscher commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1594426642
AFAIK Predictive forecasting is an optional feature introduced with https://github.com/apache/superset/pull/10324. So at the moment your current workaround is the actual intended way to enable this feature. To me this does not look like a bug, but I see two issues here:
- predictive forecasting should be mentioned in the Superset docs whether it's optional or not and how to enable it (from scratch install vs docker install)
- as predictive forecasting is baked into the UI independent of it is enabled or not, I also think it should be a default feature
Unfortunately, the integration tests for this feature are currently failing (at least for me). https://github.com/apache/superset/issues/24406
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1970013306
Ahh... [this PR](https://github.com/apache/superset/pull/10324) is the reason (and provides context).
From Updating.md:
> Facebook Prophet has been introduced as an optional dependency to add support for timeseries forecasting in the chart data API. To enable this feature, install Superset with the optional dependency `prophet` or directly `pip install fbprophet`
I think with that, we probably ought to close this out, since it's not a bug, but I'll leave it open for now for further discussion, or in hopes that someone might have some cycles to improve the documentation so others don't get tripped up on this.
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "Razikus (via GitHub)" <gi...@apache.org>.
Razikus commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1970140597
Awesome guys thanks for quick answer
For me finally it worked but it works only with time series not SQL
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] sebastianliebscher commented on issue #24357: Predictive analytics not working out of the box when using docker.
Posted by "sebastianliebscher (via GitHub)" <gi...@apache.org>.
sebastianliebscher commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1594579594
In the UI, when enabling this feature it should cleanly prompt the information "missing requirement 'prophet'". Maybe your issue is related with caching activated which it is not by default. Can you test the same without caching?
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "maudrid (via GitHub)" <gi...@apache.org>.
maudrid commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1970046598
The issue ís that,since it is not installed by default, when users try to
use the feature in a chart, the get an obscure error message.
I propose that either the chart should disabled the UI for the predication,
or should show a user friendly error message.
On Wed, Feb 28, 2024, 23:17 Evan Rusackas ***@***.***> wrote:
> Ahh... this PR <https://github.com/apache/superset/pull/10324> is the
> reason (and provides context).
>
> From Updating.md:
>
> Facebook Prophet has been introduced as an optional dependency to add
> support for timeseries forecasting in the chart data API. To enable this
> feature, install Superset with the optional dependency prophet or
> directly pip install fbprophet
>
> I think with that, we probably ought to close this out, since it's not a
> bug, but I'll leave it open for now for further discussion, or in hopes
> that someone might have some cycles to improve the documentation so others
> don't get tripped up on this.
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/superset/issues/24357#issuecomment-1970013306>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ACJQ646VRUBYIWTMYV6QITDYV6UHNAVCNFSM6AAAAAAZDMIW3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZQGAYTGMZQGY>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] maudrid commented on issue #24357: Predictive analytics not working out of the box when using docker.
Posted by "maudrid (via GitHub)" <gi...@apache.org>.
maudrid commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1594541695
If this is an optional feature, there should be a feature flag that enables it.
Having pieces of the UI just fail with uninformative errors, is just bad.
In addition to a feature flag, there should be documentation on how to enable it.
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "Razikus (via GitHub)" <gi...@apache.org>.
Razikus commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1969537280
bump
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "Razikus (via GitHub)" <gi...@apache.org>.
Razikus commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1969985335
In prod version in 3.11 it's the same
Prophet not installed
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Predictive analytics not working out of the box when using docker. [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #24357:
URL: https://github.com/apache/superset/issues/24357#issuecomment-1970052963
No disagreement, I think that's the right thing to do. We can call that a bug for now. I'd encourage anyone interested in this thread to open a PR though, as I don't think any of the core committers have this high on their list.
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org