Skip to content

Commit d8c3a12

Browse files
RomeoVclaude
andcommitted
Remove logartifact calls from tutorial and examples
Removes calls to the non-existent logartifact function that was causing UndefVarError. The function is not implemented in the current codebase. Changes: - Commented out logartifact calls with TODO reference to issue #61 - Commented out rm(plotfilename) to keep plot files available - Added clear comments explaining the temporary workaround This makes the tutorial functional while preserving the intent for future implementation when artifact upload functionality is added. Fixes #61 (temporary workaround until function is implemented) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent b5bad10 commit d8c3a12

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

docs/src/tutorial.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,11 +102,12 @@ end
102102
plotfilename = "pricepaths-plot.png"
103103
png(plotfilename)
104104

105-
# Upload the plot as an artifact associated with the MLFlow experiment's run
106-
logartifact(mlf, exprun, plotfilename)
105+
# TODO: Upload the plot as an artifact when logartifact function is implemented
106+
# See: https://github.com/JuliaAI/MLFlowClient.jl/issues/61
107+
# logartifact(mlf, exprun, plotfilename)
107108

108-
# remote temporary plot which was already uploaded in MLFlow
109-
rm(plotfilename)
109+
# Keep the plot file since artifact upload is not yet available
110+
# rm(plotfilename)
110111

111112
# complete the experiment
112113
updaterun(mlf, exprun, "FINISHED")

examples/simple-with-mlflow.jl

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,12 @@ end
5151
plotfilename = "pricepaths-plot.png"
5252
png(plotfilename)
5353

54-
# Upload the plot as an artifact associated with the MLFlow experiment's run
55-
logartifact(mlf, exprun, plotfilename)
54+
# TODO: Upload the plot as an artifact when logartifact function is implemented
55+
# See: https://github.com/JuliaAI/MLFlowClient.jl/issues/61
56+
# logartifact(mlf, exprun, plotfilename)
5657

57-
# remote temporary plot which was already uploaded in MLFlow
58-
rm(plotfilename)
58+
# Keep the plot file since artifact upload is not yet available
59+
# rm(plotfilename)
5960

6061
# complete the experiment
6162
updaterun(mlf, exprun, "FINISHED")

0 commit comments

Comments
 (0)