Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update Time Series Forecast ZenFile to ZenML 0.20.2 #46

Merged
merged 4 commits into from Oct 18, 2022
Merged

Update Time Series Forecast ZenFile to ZenML 0.20.2 #46

merged 4 commits into from Oct 18, 2022

Conversation

safoinme
Copy link
Contributor

Describe changes

Update theTime Series Forecast ZenFile to work with ZenML 0.20.2!

Pre-requisites

Please ensure you have done the following:

  • If my change requires a change to docs, I have updated the documentation accordingly.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Other (add details above)

@dagshub
Copy link

dagshub bot commented Oct 13, 2022

Join the discussion on DagsHub!

htahir1
htahir1 requested changes Oct 18, 2022
View reviewed changes
Copy link
Contributor

@htahir1 htahir1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small comments

time-series-forecast/README.md Show resolved Hide resolved
time-series-forecast/README.md Outdated Show resolved Hide resolved
@safoinme safoinme merged commit 37175fc into main Oct 18, 2022
@strickvl strickvl deleted the fix/update-time-series-zenml0.20.2 branch March 8, 2023 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@htahir1 htahir1 htahir1 requested changes

@strickvl strickvl Awaiting requested review from strickvl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@safoinme @htahir1