Pulp

Sorry, not sure. I don't have that model to test, but the `13b-q5` works with `MODEL_TYPE=LlamaCpp`. It would also be good if you specify exactly what you did to get...

@jcrsantiago to add threads just change it in `privateGPT.py` file: `llm = LlamaCpp(model_path=model_path, n_ctx=model_n_ctx, callbacks=callbacks, verbose=False, n_threads=8)` See `n_threads=` added at the end.

Oh, I just noticed (it happens on master too) that we are silently ignoring every unknown/missing file type. ```python3 def load_single_document(file_path: str) -> Document: ext = "." + file_path.rsplit(".", 1)[-1]...

> @PulpCattel IMHO a method function is written with regards to its caller, but with regards to its function/contract. If this function is reused for another reason, this is important....

> In a way, the first Ariane 5 was blown up because an overflow exception was not caught which was not necessary for Ariane 4. But they reused the code...

> Checking "two times" is also good for "Defense in Depth". There is no impact on performance here because normally that line is never reached and there is no extra...

See https://github.com/imartinez/privateGPT/issues/180 and https://github.com/imartinez/privateGPT/issues/214 This is a duplicate of many other issues.

See https://github.com/imartinez/privateGPT/issues/281 and https://github.com/imartinez/privateGPT/issues/295

Yeah I can reproduce. It was also mentioned in https://github.com/imartinez/privateGPT/issues/293 (and possibly a few other issues). It's because of this line: https://github.com/imartinez/privateGPT/blob/2f3aab9cfdc139f399387dbb90300d5a8bf8d2f1/ingest.py#L89 With `csv`, and maybe other formats too, the...

> how would a user benefit from this implementation? By using a virtualenv. I'm pretty sure most people using this repo are installing all the requirements on their base Python...