Closed aviraljain99 closed 1 year ago
Yes, this only supports only HFT engines, will need to be made explicit in the docs.
On 13 Oct 2022, at 10:50, Nicholas Lambourne @.***> wrote:
 @nicklambourne commented on this pull request.
In elpis/endpoints/model.py:
- engine = ENGINES["hft"]
- interface: Interface = app.config["INTERFACE"]
- interface.set_engine(engine) I'm just wondering, if the model isn't deciding what engine it uses, why does the engine need to change here? Is there some inbuilt assumption that only HFT models will be uploaded?
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because your review was requested.
@aviraljain99 When you address a comment in the latest commit, can you also resolve the related conversation? Keeps things clean :)
This PR is for the following features: