Skip to content

Model data type #717

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

Open
ThomAub opened this issue Apr 30, 2021 · 3 comments
Open

Model data type #717

ThomAub opened this issue Apr 30, 2021 · 3 comments

Comments

@ThomAub
Copy link

ThomAub commented Apr 30, 2021

Hello,

Will it be possible to configure model data type (same a tensor dtype) ?

Let's say it do some quantisation on my model I would like to be able to use redis AI and do inference on tensor in float16, int8.

I think most backend implement this mechanism but I don't know if it's something that would interest redisAI ?

@gkorland
Copy link
Contributor

gkorland commented May 2, 2021

Sure, RedisAI support different datatypes see TENSOR.SET command: https://oss.redislabs.com/redisai/commands/#aitensorset

@ThomAub
Copy link
Author

ThomAub commented May 2, 2021

Hello,

thank you for your answer and sorry I wasn’t very clear in my question.

In AI.TENSORSET it’s possible to add the datatype in the tensor META.
For AI.MOSELSET or AI.MOSELSTORE it’s not. Would it be something of interest to add a dtype in the model META ? This would represent the model weights datatype.

It’s not really an issue but just an idea for discussion.
I’m currently overcoming this with the tag field.

thanks again

@gkorland
Copy link
Contributor

gkorland commented May 2, 2021

We didn't think about since the models are usually holding this metadata in the graph definition

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants