kedro-mlflow icon indicating copy to clipboard operation
kedro-mlflow copied to clipboard

:sparkles: Enable running pipeline without mlflow.yml file (#328)

Open Galileo-Galilei opened this issue 2 years ago • 1 comments

Description

Close #328

Development notes

Change after_context_created hook to use default configuration instead of raising an error when no mlflow.yml file exists.

Checklist

  • [X] Read the contributing guidelines
  • [X] Open this PR as a 'Draft Pull Request' if it is work-in-progress
  • [x] Update the documentation to reflect the code changes
  • [x] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
  • [x] Add tests to cover your changes

Notice

  • [X] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

  • I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

  • I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

  • I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Galileo-Galilei avatar Jul 06 '22 21:07 Galileo-Galilei

Codecov Report

Merging #340 (efef7ec) into master (27aeb3e) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #340   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           28        28           
  Lines          900       908    +8     
=========================================
+ Hits           900       908    +8     
Impacted Files Coverage Δ
kedro_mlflow/config/kedro_mlflow_config.py 100.00% <ø> (ø)
kedro_mlflow/framework/hooks/mlflow_hook.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 27aeb3e...efef7ec. Read the comment docs.

codecov-commenter avatar Jul 06 '22 21:07 codecov-commenter