AgentOps-AI / AgentOps-AI/tokencost

refactor `model` parameter type to be enum

Aperta
#7 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
good first issue wontfix
Lingua principale
Python
Stelle
2k
Fork
105
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

We currently have error handling for when the `model` string input doesn't match our dictionary, but can eventually refactor to make it an enum that matches one of the below model names in our constants dictionary to make it even more explicit:

https://github.com/AgentOps-AI/tokencost/blob/main/tokencost/constants.py

Not too familiar with Python so I'm not sure if code editors have shortcuts for viewing definitions of enums like TypeScript does, but our code is easy enough to read that in our error messages as well as the code itself we just redirect people to the constants file

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.