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

Simplify logging the artifacts #65

Open
annmary-roy opened this issue Dec 12, 2022 · 2 comments
Open

Simplify logging the artifacts #65

annmary-roy opened this issue Dec 12, 2022 · 2 comments

Comments

@annmary-roy
Copy link
Contributor

annmary-roy commented Dec 12, 2022

Feedback - Can one common meta writer for all artifacts like model, metric, input, and output? The artifact type should be a user argument.

@annmary-roy
Copy link
Contributor Author

Each artifact type has a different characteristic that needs to be captured. So it makes that single method very heavy

@annmary-roy
Copy link
Contributor Author

Instead of three different api's log_dataset, log_metrics, log_model can we have a single log_artifact call and the function internally does the appropriate decisions.
If implemented the key point is this needs to be simple. it should not complicate the api's.
Should come up with a simple interface and reassess the feasibility
One approach is to maintain both low level interfaces and high level interfaces and based on the user preference they can choose, which one to choose.
Priority -

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant