BasedHardware / omi

AI wearables
https://omi.me
MIT License
3.63k stars 453 forks source link

Revert "Trim the unnecessary data for updating processing memory api" #813

Closed beastoin closed 1 month ago

beastoin commented 1 month ago

Reverts BasedHardware/omi#811

not work as expected, the response time looks higher ???

Summary by Entelligence.AI

github-actions[bot] commented 1 month ago

Image description Entelligence.AI

Walkthrough

This change simplifies the UpdateProcessingMemoryResponse class by directly returning the updated ProcessingMemory object. It also enhances the update_processing_memory function to return the processing memory directly, improving code readability and maintainability.

Changes

Files Summary
backend/models/processing_memory.py, backend/routers/processing_memories.py The type of the result field in the UpdateProcessingMemoryResponse class has been changed from UpdateProcessingMemory to ProcessingMemory. The update_processing_memory function now returns the processing memory directly instead of creating a new object.

πŸŽ‰πŸ€–

With every commit we make,

Our code becomes less opaque.

Simplified, clean, and neat,

Making updates a treat.

So here's to changes small and big,

Dancing a continuous dev-ops jig! πŸ•ΊπŸ’»


Uplevel your code reviews with Entelligence.AI Pro ### Entelligence.AI Pro If you like this project, please support us by purchasing the [Pro version](https://Entelligence.AI.ai). The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, Entelligence.AI Pro is free for open source projects.