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

Add notes on options for handling model dependencies #366

Merged
merged 5 commits into from
Nov 22, 2022

Conversation

ots22
Copy link
Member

@ots22 ots22 commented Nov 8, 2022

This note attempts to describe some of the options we've been considering for handling model packages and their dependencies (particularly system dependencies that can't be completely installed with pip), and some ideas about requirements we want from whatever we settle on.

The final version of this document is intended to describe the solution we have chosen, with some rationale, and the other options that weren't chosen. For now it is a collection of notes on a few of these options.

Comments and additional commits to this PR most welcome, particularly with additional ideas, extra detail on the existing ideas, thoughts, opinions, preferences etc.

Note that #325 and #347 would result in something like the second option ("pip with a 'model card'").

@edwardchalstrey1
Copy link
Contributor

My potentially controversial opinion is that if we get to the stage where the frontend successfully implements the model cards idea (described in 2.) we should go with option 3 "SaaS first" and even consider removing install functionality completely - see #346

Keep focus on problem in first section, rather than what we should do about it
@ots22 ots22 merged commit ec4a6f6 into main Nov 22, 2022
@edwardchalstrey1 edwardchalstrey1 deleted the model-dependencies-note branch January 3, 2023 13:39
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

Successfully merging this pull request may close these issues.

2 participants