e-johnstonn / GPT-Doc-Summarizer

Streamlit webapp for token-efficient summarization of long files using GPT 3.5/4
MIT License
137 stars 66 forks source link

"Use GPT-4" Checkbox Still Uses GPT-3.5-Turbo #5

Closed conroywhitney closed 8 months ago

conroywhitney commented 1 year ago

Hey -- I just noticed this use_gpt_4 still ends up setting model_name='gpt-3.5-turbo'

https://github.com/e-johnstonn/GPT-Doc-Summarizer/blob/3f8635f3d55719dbd8f7e745daf02f8b84607461/streamlit_app_utils.py#L122-L123

conroywhitney commented 1 year ago

I made a quick PR #6 if you find it helpful. Thanks again for creating this repo 😃

conroywhitney commented 1 year ago

Oh, my bad. I see that it's used for the actual summarization:

https://github.com/e-johnstonn/GPT-Doc-Summarizer/blob/3f8635f3d55719dbd8f7e745daf02f8b84607461/utils.py#L224-L226

So maybe you were using 3.5-turbo on purpose for the prompt generation. Feel free to close.

Thanks!