AgentOps-AI / AgentOps-AI/tokencost

refactor `model` parameter type to be enum

オープン
#7 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
good first issue wontfix
主要言語
Python
スター
2k
フォーク
105
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。