more dynamic s3 parameters

Dataherald / dataherald

Interact with your SQL database, Natural Language to SQL using LLMs

https://dataherald.readthedocs.io/en/latest/

Apache License 2.0

3.35k stars 234 forks source link

more dynamic s3 parameters #506

Closed dnnspaul closed 5 months ago

dnnspaul commented 5 months ago

Hey!

We started to use dataherald in our company and figured out that using something different than AWS S3 is kinda complicated and seems to be not intended by the code. This pull request enables the possibility for using something like minio, an s3-compatible selfhosted alternative.

Tell me what you think. Thanks for your work so far, we are very happy with DataHerald!

aazo11 commented 5 months ago

Thanks for your contribution @dnnspaul