You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently MLCommons is using ObjectOutputStream to serialize object (code link). But ThresholdedRandomCutForestState doesn't implement Serializable, so it's impossible to serialize via ObjectOutputStream.
AD is using protostuff to serialize. protostuff has one bug reported in AD opensearch-project/anomaly-detection#263, we fixed by adding local jar dependency as protostuff team has no clear plan to release new version.
If we use same way in MLCommons we need to add one more dependency and we need to maintain two serialization ways. So prefer to implement Serializable to make serialization easier.
The text was updated successfully, but these errors were encountered:
Currently MLCommons is using
ObjectOutputStream
to serialize object (code link). ButThresholdedRandomCutForestState
doesn't implementSerializable
, so it's impossible to serialize viaObjectOutputStream
.AD is using
protostuff
to serialize.protostuff
has one bug reported in AD opensearch-project/anomaly-detection#263, we fixed by adding local jar dependency as protostuff team has no clear plan to release new version.If we use same way in MLCommons we need to add one more dependency and we need to maintain two serialization ways. So prefer to implement
Serializable
to make serialization easier.The text was updated successfully, but these errors were encountered: