Closed gruckion closed 3 months ago
This looks awesome @gruckion 🔥
I wonder why I am still seeing $0
Tried logging the costData
but I can't be fully sure of the numbers, not all models that have a cost !=0 have been used e.g. babbage
.
I wonder why I am still seeing $0
Tried logging the
costData
but I can't be fully sure of the numbers, not all models that have a cost !=0 have been used e.g.babbage
.
What is the name of your model. The model name key is the important property. Will also need to add additional config to cover the new models you've introduced.
We can run into an issue, since the user can freely put the model name I can put OpenAI gpt 3-5
or any other combination. We will need to have this locked down to a selector instead of free form text / templates.
Hmm, in the latest version the model name is chosen from a dropdown, can we unify them together?
Hmm, in the latest version the model name is chosen from a dropdown, can we unify them together?
Perfect, I will align with this.
Updated:
Cost information is shown.
And configurations still works as expected.
import { getModelConfigurations } from "@/lib/model-config";
actions.ts
for the getConfigurationCost()
to make use of this single source of truth object.Also left a TODO comment, we can improve the UX with a toaster.
// TODO: Implement proper handling when turning off a default configuration
// - Prevent turning off the last default configuration
// - Show a toast message explaining why the action is not allowed
// - Implement logic to ensure at least one configuration is always set as default
{/* TODO: Add an "Actions" column for edit functionality */}
Oh beautiful, testing it now 😍
for(const x of y)
instead ofx.forEach
. forEach creates a new callback for each iteration and doesn't work with break/continue as well as not working with async operations.