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
Cleanup for kedro==0.19 #391
Labels
code_health
Refactoring required for performance, maintenance and adressing tech debt
Milestone
Comments
Galileo-Galilei
added a commit
that referenced
this issue
Jul 23, 2023
6 tasks
Galileo-Galilei
added a commit
that referenced
this issue
Jul 23, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Jul 23, 2023
6 tasks
Assign this to me, please. |
Hi @ShubhamZoro, sorry for the late reply! I'd love to assign this to you, but I think we should merge it to a develop branch because I may have a couple of release until 0.19 is released and I want to keep retrocompatibility. |
Galileo-Galilei
assigned Galileo-Galilei and ShubhamZoro and unassigned Galileo-Galilei
Jul 31, 2023
Do I need to change PickleDataset to PickleDataset or MemoryDataSet to MemoryDataset as well? |
5 tasks
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 22, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Oct 25, 2023
Galileo-Galilei
added
the
code_health
Refactoring required for performance, maintenance and adressing tech debt
label
Oct 25, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Nov 19, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Nov 19, 2023
…f the dataset instead of a string (#391)
Galileo-Galilei
added a commit
that referenced
this issue
Nov 21, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Nov 21, 2023
…f the dataset instead of a string (#391)
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Signed-off-by: Yolan Honoré-Rougé <yolan.honore.rouge@gmail.com>
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Signed-off-by: Yolan Honoré-Rougé <yolan.honore.rouge@gmail.com>
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Signed-off-by: Yolan Honoré-Rougé <yolan.honore.rouge@gmail.com>
Galileo-Galilei
added a commit
that referenced
this issue
Dec 19, 2023
Signed-off-by: Yolan Honoré-Rougé <yolan.honore.rouge@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
This tickets gather breaking changes for
kedro==0.19
:MlflowArtifactData**S**et
->MlflowArtifactData**s**et
-> done in Changing DataSet to Dataset#391 #439MlflowModelSaverDataSet
->MlflowLocalModelDataSet
(orMlflowModelLocalFileDataSet
?) andMlflowModelLoggerDataSet
->MlflowRemoteModelDataSet
(orMlflowModelTrackingDataSet
?) To be discussed.kedro_mlflow.io
folder tokedro_mlflow.datasets
#479kedro-datasets
and to the new datasets without the capital S version (e.g.PickleDataset
,CSVDataset
) #460catalog._data_sets
to public attribute depending on kedroConfigLoader
andTemplatedConfigLoader
ans cleanup the codebase and tests, addOmegaConfigLoader
insteadafter_context_created
hook :kedro-mlflow/kedro_mlflow/framework/hooks/mlflow_hook.py
Lines 128 to 131 in 845ad91
config_loader
as a property kedro-org/kedro#1459.The text was updated successfully, but these errors were encountered: