fix(medcat-trainer): Dont use addons in the pipeline for document processing#564
fix(medcat-trainer): Dont use addons in the pipeline for document processing#564alhendrickson wants to merge 4 commits into
Conversation
|
This isn't the best (or at least a complete) way of doing this. The other potential issue is that the current process can change the order of the addons. This shouldn't matter. But it might. Especially with MetaCAT reusing some of the data paths to avoid duplicate computations. Ideally we'd want to fix this on the core lib side:
With that said, my preferred method to remove addons would generally be something like this: That way we ensure that everything is working as expected. Each addon should have its own config so that could be used to add them back if/when needed. PS: |
MedCAT Addons are not needed when calling prepare-documents API
This fixes this error, along with any other issue found inside an addon that isn't used
This is definitely messy as it mutates the cached CAT, but seems like this is done other places anyway like cat.config.components.linking.filters.cuis = cuis