Using Artifacts within the source code of TensorInference is not a very good practice. I think we should move this functionality out of this package and remove Artifacts as a project dependency. We could move the artifacts dependency to the test project instead.
I think it becomes much cleaner if our functions expect paths to the relevant files (model, evidence, query) rather than a string that is later used in combination with artifacts.
mroavi
commented
1 year ago
Using Artifacts within the source code of TensorInference is not a very good practice. I think we should move this functionality out of this package and remove Artifacts as a project dependency. We could move the artifacts dependency to the test project instead.
I think it becomes much cleaner if our functions expect paths to the relevant files (model, evidence, query) rather than a string that is later used in combination with artifacts.