abcsys / libem

Compound AI toolchain for fast and accurate entity matching, powered by LLMs.
https://libem.org
Apache License 2.0
19 stars 3 forks source link

fix: add option to change LIBEM_SAMPLE_DATA_PATH in config.yaml #96

Closed daiwaid closed 2 months ago

daiwaid commented 2 months ago

What this PR does / why we need it:

When the libem module is installed through pip, LIBEM_SAMPLE_DATA_PATH will not point to the correct/desired location, so the option to change the path is needed.

Which issue(s) this PR addresses (if any):

Addresses #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE
zenodflow commented 2 months ago

This is good fix. Could you also add a line or two in the benchmark/README.md to inform readers whether/what/how to configure? I.e., letting them know that they should use libem configure had they installed via pip.