Skip to content

Conversation

dineshyv
Copy link
Contributor

@dineshyv dineshyv commented Nov 15, 2024

llama-stack-client memory_banks list
❯ llama-stack-client memory_banks register your_memory_bank_name --type vector --provider-id inline::faiss-0

Memory Bank Configuration:
{
│   'memory_bank_type': 'vector',
│   'chunk_size_in_tokens': 512,
│   'embedding_model': 'all-MiniLM-L6-v2',
│   'overlap_size_in_tokens': 64
}
❯ llama-stack-client memory_banks list
┏━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ identifier            ┃ provider_id     ┃ provider_resource_id  ┃ memory_bank_type ┃ params                            ┃
┡━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
│ your_memory_bank_name │ inline::faiss-0 │ your_memory_bank_name │ vector           │ chunk_size_in_tokens: 512         │
│                       │                 │                       │                  │ embedding_model: all-MiniLM-L6-v2 │
│                       │                 │                       │                  │ overlap_size_in_tokens: 64        │
│                       │                 │                       │                  │ type: memory_bank                 │
│                       │                 │                       │                  │                                   │
└───────────────────────┴─────────────────┴───────────────────────┴──────────────────┴───────────────────────────────────┘
llama-stack-client memory_banks unregister your_memory_bank_name
Memory bank 'your_memory_bank_name' deleted successfully

llama-stack-client memory_banks unregister your_memory_bank_name

llama_stack_client.BadRequestError: Error code: 400 - {'detail': 'Invalid value: Memory bank your_memory_bank_name not found'}
Screenshot 2024-11-15 at 6 30 58 PM

Copy link
Contributor

@raghotham raghotham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please memory bank type also a top level column

@dineshyv
Copy link
Contributor Author

done.

@dineshyv dineshyv changed the title memory bank update and delete memory bank unregister Nov 15, 2024
@dineshyv dineshyv force-pushed the dineshyv/memorybank-update-delete branch 3 times, most recently from d75755b to a1092f7 Compare November 15, 2024 23:25
@dineshyv dineshyv force-pushed the dineshyv/memorybank-update-delete branch from a1092f7 to 2c42d66 Compare November 15, 2024 23:26
@dineshyv dineshyv merged commit 5eb688f into main Nov 16, 2024
3 checks passed
@dineshyv dineshyv deleted the dineshyv/memorybank-update-delete branch November 16, 2024 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants