rstudio / rstudio/vetiver-python

Error earlier when people try to deploy a temporary board

Open
#193 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

deploy enhancement
Dominant language
Python
Stars
71
Forks
20
PR merge metrics
No merged PRs in 30d

Description

Currently, when people try to deploy a model from a temporary pins board, the deployment will successfully complete and then error on API startup. On deployment, raising an error that you're trying to deploy a model from a temporary board would be a more elegant failure.

One place that this might be painful is in testing situations, but I think tearing down boards after/using temporary directories would be an okay workaround.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Trace the deployment flow for models backed by temporary pins boards and identify where board type is available before deployment begins. Add an early error for temporary boards, then cover the behavior in the relevant deployment tests and verify that normal boards still deploy successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.