martaannaj / RecommenderServer

GNU General Public License v3.0
1 stars 2 forks source link

Check the traversal pointer of IItem when serialized #47

Closed miselico closed 1 year ago

miselico commented 1 year ago

Currently, the traversal pointer for IItem is not serialized. It appeared this was not necessary, but it seems this traversal pointer is used in the recommendation https://github.com/martaannaj/RecommenderServer/blob/3d0bf868b94284419b9b7b0b9a1be11451ec812f/schematree/recommendation.go#L101-L116

miselico commented 1 year ago

This is not an issue because the traversal pointer is restored while deserializing. The order in the linked lists might be different, but that does not matter for the behavior. Adding a test case covering this soon.

miselico commented 1 year ago

Done in 35f9fd5