Skip to content

Docker images with missing prophet fix #767

Merged
merged 3 commits into from
Jun 22, 2022
Merged

Docker images with missing prophet fix #767

merged 3 commits into from
Jun 22, 2022

Conversation

martins0n
Copy link
Contributor

@martins0n martins0n commented Jun 21, 2022

IMPORTANT: Please do not create a Pull Request without creating an issue first.

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Type of Change

  • Examples / docs / tutorials / contributors update
  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves an existing feature)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

@martins0n martins0n changed the title FIX: prophet Docker images with missing prophet fix Jun 21, 2022
@github-actions
Copy link

github-actions bot commented Jun 21, 2022

🚀 Deployed on https://deploy-preview-767--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request June 21, 2022 10:39 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Jun 21, 2022

Codecov Report

Merging #767 (b99bef0) into master (e40ed7a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #767   +/-   ##
=======================================
  Coverage   83.57%   83.57%           
=======================================
  Files         122      122           
  Lines        6776     6776           
=======================================
  Hits         5663     5663           
  Misses       1113     1113           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@martins0n martins0n self-assigned this Jun 21, 2022
@iKintosh iKintosh self-requested a review June 21, 2022 11:21
@iKintosh
Copy link
Contributor

Still strange why pip does not install prophet from requirement.txt file. But if that allows to fix the bug then LGTM

@iKintosh iKintosh enabled auto-merge (squash) June 22, 2022 09:57
@github-actions github-actions bot temporarily deployed to pull request June 22, 2022 09:59 Inactive
@iKintosh iKintosh merged commit d5e6c8a into master Jun 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants