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

Hot fix for backward compatiblity of kedro datasets #1600

Merged
merged 3 commits into from
Oct 24, 2023
Merged

Hot fix for backward compatiblity of kedro datasets #1600

merged 3 commits into from
Oct 24, 2023

Conversation

ravi-kumar-pilla
Copy link
Contributor

@ravi-kumar-pilla ravi-kumar-pilla commented Oct 23, 2023 •
edited
Loading

Description

Fixes the experiment tracking issue when user is on kedro viz > 6.6.0 and kedro-datasets >= 1.7.0

Development notes

NOTE: This is a hot fix and needs to be pushed before 6.6.1 release to support kedro-datasets >= 1.7.0

QA notes

  • Experiment Tracking should work with kedro-datasets < 1.7.1 and kedro-datasets >= 1.7.1

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added new entries to the RELEASE.md file
  • Added tests to cover my changes

@ravi-kumar-pilla
hot fix for backward compatiblity in kedro datasets and viz
b1b9999 
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
@ravi-kumar-pilla
upgrade plotly
1e28754 
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
@ravi-kumar-pilla
upgrading plotly as per kedro datasets
d08ad74 
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
@ravi-kumar-pilla ravi-kumar-pilla marked this pull request as ready for review October 23, 2023 20:19
@tynandebold tynandebold merged commit 7922698 into main Oct 24, 2023
22 checks passed
@tynandebold tynandebold deleted the fix/bc-kedro-datasets branch October 24, 2023 09:55
@tynandebold tynandebold mentioned this pull request Oct 24, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tynandebold tynandebold tynandebold approved these changes

@merelcht merelcht merelcht approved these changes

@rashidakanchwala rashidakanchwala Awaiting requested review from rashidakanchwala rashidakanchwala is a code owner

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

Successfully merging this pull request may close these issues.
3 participants
@ravi-kumar-pilla @tynandebold @merelcht