Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a Warning Callout Re Cost #93

Closed
ObsidianTTRPGProject opened this issue Mar 26, 2023 · 12 comments
Closed

Add a Warning Callout Re Cost #93

ObsidianTTRPGProject opened this issue Mar 26, 2023 · 12 comments

Comments

@ObsidianTTRPGProject
Copy link

Just something I think people should be pre-warned about in a louder way. The cost of using this can really ramp up quite quickly if you have a large vault. The default limit for ChatGPT max spend per month is $120. I would be warning people that they should review these settings ahead of time so they don't accidentally find themselves facing a potentially large bill. I set mine at $20 for the month and hit that in about 30 minutes. Albeit my vault is rather massive.

@brianpetro
Copy link
Owner

I set mine at $20 for the month and hit that in about 30 minutes. Albeit my vault is rather massive.

That's quite impressive. My bill for all of last month was ~$3.

There's definitely some room for improvement regarding visibility into costs. Do you have any suggestions?

@ashishbhateja
Copy link

I'm also curious about what caused such a high bill. Mine's usually only a few cents a day..

@pinuke
Copy link
Contributor

pinuke commented Mar 26, 2023

Usually high cost is linked to failed embeddings or trying to embed too many times. This can be the case with Obsidian Sync or other Obsidian Sync methods.

@ObsidianTTRPGProject
Copy link
Author

Managed to hit : $24.51 :D

It's a TTRPG Vault that I've been creating for 10+ years so lots of notes and it was created in a tool called Realm Works so it specialised in creating links... lots and lots of links. If you would like to see how much content... well I made a video a year ago as an example. https://www.youtube.com/watch?v=newKX6mQJ9M&t=2s

I'm not using Obsidian Sync. I sync via Github but I do that manually using Obsidian Desktop every so often.

I would have a callout in the Read Me advising people of these settings, something along these lines:

Warning: This plugin sends your data back to OpenAI which utilises credits. The default limit for spending with OpenAI is $120 per month. It is recommended you check these settings and set it to your preferred max monthly limit prior to enabling this plugin. The larger your vault, the more it is going to cost to 'train' the AI
https://platform.openai.com/account/billing/limits

@pinuke
Copy link
Contributor

pinuke commented Mar 26, 2023

Currently working on a PR that shows this dialog whenever you set the API key in the plugin settings:

  • additonally, I found a few tricks that you can use @brianpetro:
    • call new Obsidian.Notice( message, timeout ) with timeout=null to keep the dialog box up indefinitely. Neat little trick.
    • you can set message = document.createElement( "span" ) to insert html into the dialog box, as message can either be of type string or DocumentFragment

image

@ObsidianTTRPGProject
Copy link
Author

I have no idea if this is even possible. But some kind of training status would be amazing. Something like: X notes processed for example might be able to give people an idea of how many notes have been sent.

@pinuke
Copy link
Contributor

pinuke commented Mar 26, 2023

#96 adds this dialog and also updates the readme.

@pinuke
Copy link
Contributor

pinuke commented Mar 26, 2023

some kind of training status would be amazing

Recommend moving to a new FR (Feature Request) via Issues or that comment will likely get lost in the weeds due to this new PR lol.

@brianpetro
Copy link
Owner

@ObsidianTTRPGProject are you sure that your high bill isn't from this other plugin that's giving you the errors submitted in #99 ?

@ObsidianTTRPGProject
Copy link
Author

It absolutely got me questioning everything right now. I'll be doing some testing today and get back to you. Apologies for the confusion. I thought I'd turned that one off.

@brianpetro
Copy link
Owner

Closing as platforms are increasing usage visibility 🌴

@brianpetro
Copy link
Owner

Recorded response https://youtu.be/DS5pUHa2UCk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants