MathVista icon indicating copy to clipboard operation
MathVista copied to clipboard

Published 20 hours ago • verified publisher icon lupantech

Redundant implementations of `get_chat_response`

Open mattmazzola opened this issue 1 year ago • 0 comments

There is an implementation in utilities#get_chat_response and models/gpt#get_response. These could be unified

https://github.com/lupantech/MathVista/blob/82f68d09b4cbffe9d0dfd7542c599810e30c9a99/utilities.py#L159-L199

https://github.com/lupantech/MathVista/blob/82f68d09b4cbffe9d0dfd7542c599810e30c9a99/models/gpt.py#L16-L55

mattmazzola avatar Feb 12 '24 19:02 mattmazzola