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 0.18.2 #1695

Merged
merged 14 commits into from
Jul 8, 2022
Merged

Release 0.18.2 #1695

merged 14 commits into from
Jul 8, 2022

Conversation

AhdraMeraliQB
Copy link
Contributor

@AhdraMeraliQB AhdraMeraliQB commented Jul 7, 2022

Description

Release 0.18.2

Major features and improvements

  • Added abfss to list of cloud protocols, enabling abfss paths.
  • Kedro now uses the Rich library to format terminal logs and tracebacks.
  • The file conf/base/logging.yml is now optional. See our documentation for details.
  • Introduced a kedro.starters entry point. This enables plugins to create custom starter aliases used by kedro starter list and kedro new.
  • Reduced the kedro new prompts to just one question asking for the project name.

Bug fixes and other changes

  • Bumped pyyaml upper bound to make Kedro compatible with the pyodide stack.
  • Updated project template's Sphinx configuration to use myst_parser instead of recommonmark.
  • Reduced number of log lines by changing the logging level from INFO to DEBUG for low priority messages.
  • Kedro's framework-side logging configuration no longer performs file-based logging. Hence superfluous info.log/errors.log files are no longer created in your project root, and running Kedro on read-only file systems such as Databricks Repos is now possible.
  • The root logger is now set to the Python default level of WARNING rather than INFO. Kedro's logger is still set to emit INFO level messages.
  • SequentialRunner now has consistent execution order across multiple runs with sorted nodes.
  • Bumped the upper bound for the Flake8 dependency to <5.0.
  • kedro jupyter notebook/lab no longer reuses a Jupyter kernel.
  • Required cookiecutter>=2.1.1 to address a known command injection vulnerability.
  • The session store no longer fails if a username cannot be found with getpass.getuser.
  • Added generic typing for AbstractDataSet and AbstractVersionedDataSet as well as typing to all datasets.
  • Rendered the deployment guide flowchart as a Mermaid diagram, and added Dask.

Minor breaking changes to the API

  • The module kedro.config.default_logger no longer exists; default logging configuration is now set automatically through kedro.framework.project.LOGGING. Unless you explicitly import kedro.config.default_logger you do not need to make any changes.

Upcoming deprecations for Kedro 0.19.0

  • kedro.extras.ColorHandler will be removed in 0.19.0.

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 a description of this change in the RELEASE.md file
  • Added tests to cover my changes

AhdraMeraliQB and others added 2 commits July 7, 2022 13:49
Signed-off-by: Ahdra Merali <ahdra.merali@quantumblack.com>
@AhdraMeraliQB AhdraMeraliQB changed the title Release/0.18.2 Release 0.18.2 Jul 7, 2022
CITATION.cff Show resolved Hide resolved
Comment on lines 72 to 73
After that you can use this starter with `kedro new --starter=test_plugin_starter --checkout=main`.

Copy link
Contributor

@antonymilne antonymilne Jul 8, 2022

Choose a reason for hiding this comment

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

Suggested change
After that you can use this starter with `kedro new --starter=test_plugin_starter --checkout=main`.
After that you can use this starter with `kedro new --starter=test_plugin_starter`.
```{note}
If your starter lives on a git repository, by default Kedro attempts to use a tag or branch labelled with your version of Kedro, e.g. `0.18.2.`. This means that you can host different versions of your starter template on the same repository, and the correct one will automatically be used. If you do not wish to follow this structure, you should override it with the `checkout` flag, e.g. `kedro new --starter=test_plugin_starter --checkout=main`.
```

@noklam I think this is clearer?

I've actually always thought (and I think @Galileo-Galilei agrees) that we should change this behaviour since in reality no one maintains a starters repo which they keep up to date with every single version of the kedro template.

  • by default don't try and checkout the kedro version unless it's an official starter (cookiecutter will by default look for the right main/master branch if you don't specify checkout)
  • and/or enable people to specify checkout in their KedroStarterSpec

But that's a discussion for another time.

@AhdraMeraliQB AhdraMeraliQB marked this pull request as ready for review July 8, 2022 09:11
@antonymilne
Copy link
Contributor

antonymilne commented Jul 8, 2022

I think we should do the change from kedro==0.18.2 to kedro~=0.18.2 in this PR as well. Also @noklam was planning to alter the test starter plugin e2e test a bit - please can you wait for that?

Edit: actually the e2e test isn't urgent tbh... I just thought it would be easiest to stick it in this PR also given the e2e test change is already here.

AhdraMeraliQB and others added 7 commits July 8, 2022 10:30
Signed-off-by: Ahdra Merali <ahdra.merali@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
…ase/0.18.2

Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

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

image

Only joking of course 😉 Looks good to me!

Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
@noklam
Copy link
Contributor

noklam commented Jul 8, 2022

image

Copy link
Contributor

@SajidAlamQB SajidAlamQB left a comment

Choose a reason for hiding this comment

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

release meme
🎉

Signed-off-by: Ahdra Merali <ahdra.merali@quantumblack.com>
@AhdraMeraliQB AhdraMeraliQB merged commit b04ffe5 into main Jul 8, 2022
@AhdraMeraliQB AhdraMeraliQB deleted the release/0.18.2 branch July 8, 2022 11:45
@AhdraMeraliQB AhdraMeraliQB restored the release/0.18.2 branch July 8, 2022 15:37
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.

6 participants