-
Notifications
You must be signed in to change notification settings - Fork 112
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.
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
Release v9.2.0 #2001
Release v9.2.0 #2001
Conversation
Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Sajid Alam <[email protected]>
…into release/9.2.0 Signed-off-by: Sajid Alam <[email protected]>
Hi @SajidAlamQB , Thank you for the release PR. Some PRs worth mentioning in the Release -
I hope there will be some update to the reminder content. Thank you |
Signed-off-by: Sajid Alam <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Sajid Alam <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work team! 🎉
Hi Team, When I refresh viz after selecting a different pipeline, it resets back to default. I would like if someone can test this behavior (pid resetting to default on refresh) and confirm. I think this is a bug (v9.0.0 works fine. The issue started in v9.1.0). Thank you cc: @jitu5 |
@ravi-kumar-pilla its already on demo site, and Huong already created bug ticket on this #1997 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you @SajidAlamQB
Signed-off-by: Sajid Alam <[email protected]>
i tested the branch locally and all the UI changes are working as expect. Great job team, looking forward to releasing to our users ❤️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- If we can do so, could we add some formatting to the text to make certain key words bold
'enable' , 'disable', .viz etc. - Also, could we add link to the documentation to show how they can disable in CLI and in UI
- Also link to npm readme file for React Component
- The session store (a SQLite database) and the stats file are now conditionally
- to store session data and stat files?
What do you mean by these two points? |
Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Sajid Alam <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. thanks @SajidAlamQB
Description
Release 9.2.0
Major features and improvements
metadataPanel
indisplay
prop in Kedro-Viz react component. (IntroducemetadataPanel
prop in Kedro-Viz react component #1965)options
prop while using Kedro-Viz as a React component. (Exposing internal state via Kedro Viz React component props #1969)Bug fixes and other changes
packaging
pin in requirements. (Update Packaging Pin to >=23.0 #1947)toposort
tographlib
. (Replace toposort with graphlib #1942)isPrettyName
toggle to False by Default. (SetisPrettyName
to False by Default #1958)-p
argument option forkedro viz
to align with kedro pipeline options. (added "-p" cli option to matchkedro
pipeline options #1961)data-heap-event
with standardiseddata-test
for Cypress and Telemetry. (Refactor: Replacedata-heap-event
with standardizeddata-test
for Cypress and Telemetry #1995)Community contributions
Many thanks to the following Kedroids for contributing PR to this release:
Checklist
RELEASE.md
file