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

Consider removing micropackaging #3750

Closed
astrojuanlu opened this issue Mar 28, 2024 · 3 comments
Closed

Consider removing micropackaging #3750

astrojuanlu opened this issue Mar 28, 2024 · 3 comments

Comments

@astrojuanlu
Copy link
Member

There has been a long time milestone to assess the micropackaging workflow https://github.com/kedro-org/kedro/milestone/21

It is somewhat inconsistent with kedro package #1536, relies on binary distributions aka wheels, it's difficult to test, and most importantly not many people use it.

Are you a user of micropackaging? If so, please drop a comment below with your thoughts 👇🏽

SELECT
  COUNT(*) AS count,
  MAIN_COMMAND,
FROM HEAP_KEDRO_APP.HEAP.ANY_COMMAND_RUN
WHERE TIME > '2023-01-01'
GROUP BY MAIN_COMMAND
ORDER BY count DESC
COUNT MAIN_COMMAND
2142002
2007807 run
275495 *****
84696 viz
13073 jupyter
8127 ipython
4968 test
4915 pipeline
4349 docker
2959 templar
2253 mlflow
2026 registry
1979 package
1882 lint
1745 vertexai
1186 info
1151 azureml
1114 benchmark
909 kedro
839 mlrun
672 catalog
623 --version
595 sagemaker
590 airflow
567 -h
391 new
378 build-reqs
319 micropkg
315 fast-api
281 build-docs
237 boot
213 create
188 --help
131 kubeflow
102 azure
101 run-azure
69 --pipeline

The proposal is to:

  1. Leave this issue open for another release cycle and request feedback on our usual channels.
  2. If we don't see enough evidence that we should keep this functionality, deprecate it for 0.19. Every time a user uses kedro micropkg a DeprecationWarning should be shown (or equivalent, see discussion about FutureWarning and DeprecationWarning in various places, for example Avoid setting warning filters globally #2744 (comment))
  3. Move it to a plugin and immediately archive it. This way folks have a way to transition, we signal that we don't intend to develop it anymore in its current form, and the community can fork if someone wants to.

The alternative to (3) would be to not do anything, just deprecate and remove.

Thoughts?

@datajoely
Copy link
Contributor

I really like option 3

@merelcht
Copy link
Member

I would go for 1 + 2, 3 seems like a lot of unnecessary effort building and releasing a plugin for something we think is not used.

@astrojuanlu
Copy link
Member Author

We got no indication from users that they want us to keep this. Therefore I consider we don't have enough evidence that maintaining this feature is worth the effort.

I will open follow-up issues to deprecate, and eventually remove, this functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Status: Shipped 🚀
Development

No branches or pull requests

3 participants