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

Add MLJFlow for integration with MLflow logging platform #1034

Merged
merged 2 commits into from
Aug 24, 2023
Merged

Conversation

ablaom
Copy link
Member

@ablaom ablaom commented Aug 24, 2023

A manual page "Logging Workflows" is also provided.

cc @pebeto

@ablaom
Copy link
Member Author

ablaom commented Aug 24, 2023

@pebeto Can you please review?

@ablaom ablaom requested review from giordano and removed request for giordano August 24, 2023 01:32
@codecov-commenter
Copy link

codecov-commenter commented Aug 24, 2023

Codecov Report

Merging #1034 (ac8eb97) into dev (59df505) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1034   +/-   ##
=======================================
  Coverage   60.97%   60.97%           
=======================================
  Files           2        2           
  Lines          41       41           
=======================================
  Hits           25       25           
  Misses         16       16           
Files Changed Coverage Δ
src/MLJ.jl 100.00% <ø> (ø)

@pebeto pebeto self-requested a review August 24, 2023 02:01
Copy link
Member

@pebeto pebeto left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@ablaom ablaom merged commit 0adfa8a into dev Aug 24, 2023
3 checks passed
This was referenced Aug 24, 2023
@pebeto pebeto mentioned this pull request Aug 27, 2023
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.

3 participants