kedro-org / kedro-viz

Visualise your Kedro data and machine-learning pipelines and track your experiments.
https://demo.kedro.org
Apache License 2.0
646 stars 106 forks source link

Fix messaging level when package compatibility is not satisfied #1877

Closed ravi-kumar-pilla closed 1 month ago

ravi-kumar-pilla commented 2 months ago

Description

We check for 2 package compatibilities when running kedro viz run and if the satisified versions are not available we display an error message on the console (based on users debug level). We should make the error message as a warning instead and display more useful message

Context

Image

Steps to Reproduce

Expected Result

Actual Result

Your Environment

Include as many relevant details as possible about the environment you experienced the bug in:

Checklist

ravi-kumar-pilla commented 1 month ago

Hi @rashidakanchwala , While I started working on this, I feel we should re-visit the package compatibilities approach. At this moment, we also support CustomDataset preview and it does not seem correct to have kedro-datasets as a required package. And also, we need to see if fsspec is a mandatory requirement (apart from sqlite store, it is mostly used conditionally and in deployers). For this ticket, I will adjust the message level and change the text. Thank you