AgentOps-AI / AgentOps-AI/tokencost

refactor `model` parameter type to be enum

Aberta
#7 0 comentários 0 reações 0 responsáveis Ver no GitHub
good first issue wontfix
Linguagem predominante
Python
Estrelas
2k
Forks
105
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

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

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.