CogStack / MedCAT

Medical Concept Annotation Tool
Other
454 stars 105 forks source link

Fixing bug for metacat #474

Closed shubham-s-agarwal closed 3 months ago

shubham-s-agarwal commented 3 months ago

Fix issues with compute_class_weights JSON serialization and enforce fc2 usage when fc3 is enabled

shubham-s-agarwal commented 3 months ago

We could write a test, but then it would make sense to perhaps test every combination (of config variations). Do we want to do that now?

mart-r commented 3 months ago

We could write a test, but then it would make sense to perhaps test every combination (of config variations). Do we want to do that now?

You can do that if you like. But if you do, then I wouldn't recommend creating a separate test method for each configuration, but rather generate the various combinations of config values automatically and go through them one by one, probably with a subTest.

But then again, if you don't believe it's something we need to test, then feel free to omit.

shubham-s-agarwal commented 3 months ago

I went through the combinations we would test, and I feel we can omit the testing as previously we tested the main module (two phase learning)