Yong Bang Xiang
09/01/2023, 9:10 AMkedro-viz
does not recognise kedro-mlflow
matplotlib plots:
while it does appear as a data node in the kedro-viz
dashboard, the content cannot be examined as a usual matplotlib png.
example_plot:
type: kedro_mlflow.io.artifacts.MlflowArtifactDataSet
data_set:
type: matplotlib.MatplotlibWriter
filepath: data/08_reporting/example_plot.png
versioned: true
Nok Lam Chan
09/01/2023, 9:15 AMNero Okwa
09/01/2023, 9:29 AMDeepyaman Datta
09/01/2023, 9:33 AMMlflowArtifactDataSet
. I've never tried this, but intuitively I wouldn't expect Kedro-Viz to support a dataset using MatplotlibWriter
under the hood.Nero Okwa
09/01/2023, 9:39 AMsrc/requirements.txt
file? like this:
kedro-datasets[matplotlib.MatplotlibWriter]~=1.1
seaborn~=0.12.1
Yong Bang Xiang
09/01/2023, 9:44 AMNero Okwa
09/01/2023, 9:46 AMYong Bang Xiang
09/01/2023, 9:56 AMexample_plot:
type: matplotlib.MatplotlibWriter
filepath: data/08_reporting/example_plot.png
example_plot:
type: kedro_mlflow.io.artifacts.MlflowArtifactDataSet
data_set:
type: matplotlib.MatplotlibWriter
filepath: data/08_reporting/example_plot.png
Yolan Honoré-Rougé
09/01/2023, 10:09 AMNero Okwa
09/01/2023, 10:15 AMYolan Honoré-Rougé
09/01/2023, 10:31 AMNok Lam Chan
09/01/2023, 11:24 AMDataFrame
preview feature.
Questions:
1. Do we want to add support for individual datasets when requested?
2. Or can we provide a way that allow user to register their own dataset to be displayed?Yolan Honoré-Rougé
09/01/2023, 11:28 AMNok Lam Chan
09/01/2023, 11:28 AMpreview
feature, it is less restricted because we inspect DataFrame
instead of pandas.CSVDataSet
, meaning that if user implement their own pandas.MyDataSet
it will still work as long as it returns a DataFrame.Deepyaman Datta
09/01/2023, 11:53 AMJuan Luis
09/01/2023, 2:36 PMI think, rather than adding support for individual datasets, it may make sense to create an interface that datasets can implement to have this support+100, maybe even checking for
_repr_png_
method, like IPython/Jupyter do