nickna / Neighborly

An open-source vector database
MIT License
9 stars 2 forks source link

fix: `MemoryMappedList.Add(vector)` after other operations uses correct position #53

Closed hangy closed 2 weeks ago

hangy commented 2 weeks ago

## 📝 Description

MemoryMappedList.Add(vector) always assumed the underlying Streams to be at the correct position. There could be several cases where this was false, for example after performing LINQ operations on the MemoryMappedList.

🔗 Related Issues

Nada

💡 Additional Notes

I used the develop branch as the base, so that I had a place to put the test method in. Theoretically, this could also be done by just cherry-picking e4cb4232313898b87d9a86fe9adddec3e6e28337 to master.