zavolokas / InpaintingSite

Web app for inpainting
GNU Affero General Public License v3.0
5 stars 4 forks source link

Extend REST API to accept settings #7

Open zavolokas opened 4 years ago

zavolokas commented 4 years ago

Currently library supports some set of settings for image processing but it is not exposed via REST API. It could be beneficial to execute the processing using different settings and allow user to select a result he prefers.

ferib commented 3 years ago

I will look into this, see if I can add an API that submits configuration in JSON format and apply then on every request made by that client (based on IP?).

zavolokas commented 3 years ago

I will look into this, see if I can add an API that submits configuration in JSON format and apply then on every request made by that client (based on IP?).

2 @ferib: It's better to keep it stateless and don't store settings per client(especially per IP since one IP can be used per many different users)

The idea is to have a possibility to override existing default settings when making a request to inpaint an image.

ferib commented 3 years ago

@zavolokas Oh yeah, I can just set the settings clientside (local storage) and then send those settings over, together with the image when doing the inpainting request.