Skip to content

fix: remove duplicate 'rrf' mode in docstring#97

Open
fuleinist wants to merge 2 commits intoEverMind-AI:mainfrom
fuleinist:fix/issue-50-rrf-mode
Open

fix: remove duplicate 'rrf' mode in docstring#97
fuleinist wants to merge 2 commits intoEverMind-AI:mainfrom
fuleinist:fix/issue-50-rrf-mode

Conversation

@fuleinist
Copy link

Summary

Remove duplicate 'rrf' retrieval mode entry in demo/utils/simple_memory_manager.py docstring.

Changes

Removed the duplicate line:

- "rrf": Keyword + Vector + RRF fusion

The mode was listed twice which caused confusion.

Fixes #50

fuleinist added 2 commits February 26, 2026 17:19
- Removed ./docker/mongodb/init volume mount (directory doesn't exist)
- Changed config.json root from hardcoded /Users/admin/Desktop to .
The 'rrf' mode was listed twice in the docstring, causing confusion.
Removed the duplicate entry.
Jah-yee added a commit to Jah-yee/EverMemOS that referenced this pull request Mar 2, 2026
In simple_memory_manager.py search() method docstring,
the "rrf" retrieval mode was listed twice.

Removed the first occurrence as the second is more descriptive.

Fixes EverMind-AI#97
Jah-yee added a commit to Jah-yee/EverMemOS that referenced this pull request Mar 3, 2026
…e docstring

- Replace bare 'except:' with 'except Exception:' in:
  - src/biz_layer/mem_db_operations.py (line 146)
  - src/infra_layer/adapters/out/search/repository/episodic_memory_milvus_repository.py (line 127)

- Enforce ISO 8601 timestamp format in episodic memory prompts:
  - Add strict format requirements: YYYY-MM-DDTHH:MM:SSZ or YYYY-MM-DDTHH:MM:SS+HH:MM
  - Update examples to use ISO 8601 format
  - Prevents inconsistent timestamp formats (Chinese chars, weekdays, AM/PM, UTC)

- Remove duplicate 'rrf' entry in demo/utils/simple_memory_manager.py docstring

Fixes EverMind-AI#107, EverMind-AI#97, EverMind-AI#108 (related to EverMind-AI#48)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] two rrf mode in demo code

1 participant