Make `vetiver_post` extensible for all endpoints #130

isabelizimm · 2022-11-21T22:26:30Z

removes the creation of a specific /predict, rather, will use arguments in vetiver_post
refactor tests to reflect this

into multiple-api

vetiver/tests/test_pytorch.py

vetiver/tests/test_add_endpoint.py

vetiver/server.py

isabelizimm added 6 commits November 16, 2022 13:55

remove hardcoded /predict endpoint

cd8d25a

playing around with jsonable_encoder

58e6524

remove hardcoded /predict endpoint

ec4d09a

playing around with jsonable_encoder

74d3931

Merge branch 'multiple-api' of https://github.com/rstudio/vetiver-python

eeb6bba

into multiple-api

TST: rebase and refactor testing

8b0489f

isabelizimm commented Nov 21, 2022

View reviewed changes

vetiver/tests/test_pytorch.py Show resolved Hide resolved

vetiver/tests/test_add_endpoint.py Show resolved Hide resolved

ganesh-k13 reviewed Nov 22, 2022

View reviewed changes

vetiver/server.py Show resolved Hide resolved

vetiver/server.py Outdated Show resolved Hide resolved

use urljoin for paths

26a6477

isabelizimm marked this pull request as ready for review November 28, 2022 22:28

isabelizimm requested a review from has2k1 November 28, 2022 22:29

has2k1 reviewed Nov 29, 2022

View reviewed changes

vetiver/server.py Outdated Show resolved Hide resolved

isabelizimm added 2 commits November 29, 2022 14:35

updates from review

af60e83

update imports

4bdd479

isabelizimm merged commit 9a93379 into main Nov 29, 2022

isabelizimm deleted the multiple-api branch November 29, 2022 20:07

Provide feedback