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.

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 v10.1.0 #2199

Merged
merged 9 commits into from
Nov 21, 2024
Merged

Release v10.1.0 #2199

merged 9 commits into from
Nov 21, 2024

Conversation

jitu5
Copy link
Contributor

@jitu5 jitu5 commented Nov 20, 2024

Description

Release 10.1.0

Major features and improvements

Bug fixes and other changes

Community contributions

Many thanks to the following Kedroids for contributing PR to this release:

update-reminder-content.js content

Screenshot 2024-11-21 at 12 27 31 p m

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

@jitu5 jitu5 self-assigned this Nov 20, 2024
RELEASE.md Outdated Show resolved Hide resolved
@astrojuanlu
Copy link
Member

Should we remove the reFocus entry from the UI release highlights?

@Huongg
Copy link
Contributor

Huongg commented Nov 21, 2024

hey @jitu5 I tested it locally and all functionality working as expected, but I feel like the in release note we miss quite a bit of big work we did for this release too. Those included 1 bug reported by our user.

@jitu5
Copy link
Contributor Author

jitu5 commented Nov 21, 2024

hey @jitu5 I tested it locally and all functionality working as expected, but I feel like the in release note we miss quite a bit of big work we did for this release too. Those included 1 bug reported by our user.

@Huongg

let me add above release notes under "Bug fixes and other changes"

Copy link
Contributor

@rashidakanchwala rashidakanchwala left a comment

Choose a reason for hiding this comment

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

I haven't QA-d it yet. Release notes should contain all the information (user facing as well as non facing). The Update Reminder section should only contain user changing top 4/5 updates.

I am not sure if we should add Opt out telemetry, as they of opt out from the Kedro project and this was launched on Kedro sometime back.

Below are my thoughts on what to add for Update Reminder section

  • Support for Python 3.12
    Kedro-Viz now supports Python 3.12, ensuring compatibility with the latest Python version.

  • Consistent Flowchart Positioning
    Flowchart positions now remain consistent across renders. Previously, the flowchart layout would change every time users made changes to their Kedro project, especially when using the --autoreload function. This issue has been resolved, and the layout will only change for significant node or pipeline updates.

  • Smarter --autoreload File Watcher
    The --autoreload watcher has been optimized to monitor only relevant files, reducing unnecessary reloads. It now excludes files listed in the user’s .gitignore and focuses on key files like .py and .yaml.

  • Improved Port Management
    Kedro-Viz will now open on a new port if the default port 4141 is already in use. This fixes an issue where running multiple instances of Kedro-Viz caused one to overwrite the other. For example, if 4141 is occupied, the next instance will use 4142, and so on.

  • Fixed Tag Undefined Issue
    Resolved a bug where the tag was occasionally undefined when pipelines were ordered differently in the pipeline registry. This was a user-reported issue, and it's now fully addressed.

RELEASE.md Outdated Show resolved Hide resolved
RELEASE.md Outdated Show resolved Hide resolved
@rashidakanchwala
Copy link
Contributor

rashidakanchwala commented Nov 21, 2024

Besides the one @Huongg mentioned, #2185, #2113, #2181, #2183, #2144 (Refactor Flowchart Models), #2139, #2124, #2110, #2095 (Refactor CSS Z-index) is missing,

Signed-off-by: Jitendra Gundaniya <[email protected]>
Copy link
Contributor

@rashidakanchwala rashidakanchwala left a comment

Choose a reason for hiding this comment

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

Look's good! There's a lot of changes this release. Fingers crossed, all goes well.

Copy link
Contributor

@Huongg Huongg left a comment

Choose a reason for hiding this comment

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

LGTM too @jitu5, thanks for adding the changes

@SajidAlamQB
Copy link
Contributor

Tested manually and all looks good to me, thank you @jitu5!

Copy link
Contributor

@ravi-kumar-pilla ravi-kumar-pilla left a comment

Choose a reason for hiding this comment

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

LGTM !! Great work team 💯

Copy link
Member

@astrojuanlu astrojuanlu left a comment

Choose a reason for hiding this comment

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

Let's go 🚀

@jitu5 jitu5 merged commit eac8d39 into main Nov 21, 2024
39 checks passed
@jitu5 jitu5 deleted the release/10.1.0 branch November 21, 2024 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants