ConvE overrides the EmbeddingModel _initialize_early_stopping() function, as there are some different requirements for the OneToNAdapter - however the early stopping params 'corrupt_side' was not copied into the eval_config, which has now been fixed.
ConvE documentation said that early_stopping with 's', 's,o' corruption strategies was supported, but it wasn't. Getting ConvE to work would require over-riding the current EmbeddingModel initialize_early_stopping function. The recommended setting for ConvE early stopping was already object corruptions, and making the changes necessary for subject corruptions is needless overhead and code duplication for an edge case, so now ConvE will throw an error if subject corruptions of any kind are specified in early stopping.
ConvE overrides the EmbeddingModel
_initialize_early_stopping()function, as there are some different requirements for the OneToNAdapter - however the early stopping params 'corrupt_side' was not copied into the eval_config, which has now been fixed.ConvE documentation said that early_stopping with 's', 's,o' corruption strategies was supported, but it wasn't. Getting ConvE to work would require over-riding the current EmbeddingModel
initialize_early_stoppingfunction. The recommended setting for ConvE early stopping was already object corruptions, and making the changes necessary for subject corruptions is needless overhead and code duplication for an edge case, so now ConvE will throw an error if subject corruptions of any kind are specified in early stopping.