-
Notifications
You must be signed in to change notification settings - Fork 966
Add Russian translations for README and graph visualization #802
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Please make sure all the checkboxes are checked:
|
WalkthroughA new Russian-language README file was added to the Cognee project within the community directory. This documentation introduces the platform, its functionalities, and usage instructions, including installation, basic usage, architecture diagrams, and contributor acknowledgments. The README also provides links to related resources and encourages community contributions. Additionally, the main README.md was updated to include a link to the new Russian README under the "Available Languages" section. No changes were made to code or public entities. Changes
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
🧰 Additional context used🪛 LanguageToolcommunity/README.ru.md[uncategorized] ~86-~86: Деепричастия в одном предложении должны быть одинакового вида и стоять в одинаковом времени (DPT_Unification) 🪛 markdownlint-cli2 (0.17.2)community/README.ru.md118-118: Fenced code blocks should have a language specified (MD040, fenced-code-language) 🔇 Additional comments (5)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (2)
community/README.ru.md (2)
117-120: Specify language for the fenced code block
The output snippet uses a fenced block without a language identifier. Adding one (e.g.,text) will improve syntax highlighting and readability.Apply this diff:
--- a/community/README.ru.md +++ b/community/README.ru.md @@ -117,4 +117,4 @@ Пример вывода: -``` +```text Обработка естественного языка (NLP) — это междисциплинарная область, которая объединяет компьютерные науки и информационный поиск. Она включает в себя технологии и методы обработки человеческого языка для создания интерфейсов и обработки данных. -``` +```🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
118-118: Fenced code blocks should have a language specified
null(MD040, fenced-code-language)
86-87: Improve participle usage for clarity
The phrase on line 86 combines two participles of different types (“создав” и “используя”), which can be stylistically awkward in Russian. Consider rephrasing for a smoother construction.Example rewrite:
-Вы также можете установить переменные, создав файл .env, используя наш <a href="https://github.com/topoteretes/cognee/blob/main/.env.template">шаблон</a>. +Вы также можете создать файл `.env` на основе нашего шаблона (<a href="https://github.com/topoteretes/cognee/blob/main/.env.template">.env.template</a>), чтобы задать переменные окружения.🧰 Tools
🪛 LanguageTool
[uncategorized] ~86-~86: Деепричастия в одном предложении должны быть одинакового вида и стоять в одинаковом времени
Context: ... Вы также можете установить переменные, создав файл .env, используя наш <a href="https://github.com/topoter...(DPT_Unification)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
community/graph_visualization_ru.pngis excluded by!**/*.png,!**/*.png
📒 Files selected for processing (1)
community/README.ru.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
community/README.ru.md
[uncategorized] ~86-~86: Деепричастия в одном предложении должны быть одинакового вида и стоять в одинаковом времени
Context: ... Вы также можете установить переменные, создав файл .env, используя наш <a href="https://github.com/topoter...
(DPT_Unification)
🪛 markdownlint-cli2 (0.17.2)
community/README.ru.md
118-118: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
| <a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a> | ||
| [Открыть пример графа в браузере](https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html). | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Ensure the graph visualization image path is correct
The src="graph_visualization_ru.png" reference is relative, but the asset likely lives under /assets. Without a full URL or correct relative path, the image won’t display.
Mirror the English README pattern:
-<a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a>
+<a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="https://raw.githubusercontent.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a>Please verify that graph_visualization_ru.png exists in the assets directory on the main branch.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a> | |
| [Открыть пример графа в браузере](https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html). | |
| <a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="https://raw.githubusercontent.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a> | |
| [Открыть пример графа в браузере](https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html). |
🤖 Prompt for AI Agents (early access)
In community/README.ru.md around lines 123 to 125, the image source path for the graph visualization is relative and likely incorrect. Update the src attribute to use the full relative path pointing to the assets directory, for example, src="assets/graph_visualization_ru.png", to ensure the image loads correctly. Confirm that the image file exists in the assets folder on the main branch before updating the path.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
♻️ Duplicate comments (1)
community/README.ru.md (1)
123-123: Ensure the graph visualization image path is correct
The imagegraph_visualization_ru.pngis referenced relatively and may not resolve unless placed alongside this README. Mirror the English README by pointing to the asset in the repository.-<a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a> +<a href="https://rawcdn.githack.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization.html"><img src="https://raw.githubusercontent.com/topoteretes/cognee/refs/heads/main/assets/graph_visualization_ru.png" width="100%" alt="Визуализация графа"></a>
🧹 Nitpick comments (1)
community/README.ru.md (1)
3-3: Unify branch references for the logo asset
The Russian README references thedevbranch for the Cognee logo, while other assets (e.g., benefits image) usemain. To keep consistency with the primary documentation and ensure the asset path remains valid, consider switching tomain.- <img src="https://raw.githubusercontent.com/topoteretes/cognee/refs/heads/dev/assets/cognee-logo-transparent.png" alt="Cognee Logo" height="60"> + <img src="https://raw.githubusercontent.com/topoteretes/cognee/refs/heads/main/assets/cognee-logo-transparent.png" alt="Cognee Logo" height="60">
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
community/graph_visualization_ru.pngis excluded by!**/*.png,!**/*.png
📒 Files selected for processing (2)
README.md(1 hunks)community/README.ru.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- README.md
🧰 Additional context used
🪛 LanguageTool
community/README.ru.md
[uncategorized] ~86-~86: Деепричастия в одном предложении должны быть одинакового вида и стоять в одинаковом времени
Context: ... Вы также можете установить переменные, создав файл .env, используя наш <a href="https://github.com/topoter...
(DPT_Unification)
| 1. Что такое память ИИ: | ||
| [Видео](https://github.com/user-attachments/assets/8b2a0050-5ec4-424c-b417-8269971503f0) | ||
|
|
||
| 2. Простая демонстрация GraphRAG | ||
| [Видео](https://github.com/user-attachments/assets/d80b0776-4eb9-4b8e-aa22-3691e2d44b8f) | ||
|
|
||
| 3. Cognee с Ollama | ||
| [Видео](https://github.com/user-attachments/assets/8621d3e8-ecb8-4860-afb2-5594f2ee17db) | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Verify demonstration video URLs
The demo section links to assets under github.com/user-attachments. Please confirm these URLs are publicly accessible and correct, or replace them with direct links in the assets/ directory.
🏁 Script executed:
#!/bin/bash
# Check HTTP status for each demo asset
urls=(
"https://github.com/user-attachments/assets/8b2a0050-5ec4-424c-b417-8269971503f0"
"https://github.com/user-attachments/assets/d80b0776-4eb9-4e...
"https://github.com/user-attachments/assets/8621d3e8-ecb8-4860-afb2-5594f2ee17db"
)
for url in "${urls[@]}"; do
if curl -I --silent "$url" | grep -q "200 OK"; then
echo "$url is reachable"
else
echo "⚠️ $url is NOT reachable"
fi
doneLength of output: 226
🏁 Script executed:
#!/bin/bash
# This script checks the HTTP status of each demo asset URL to ensure they're publicly accessible.
urls=(
"https://github.com/user-attachments/assets/8b2a0050-5ec4-424c-b417-8269971503f0"
"https://github.com/user-attachments/assets/d80b0776-4eb9-4b8e-aa22-3691e2d44b8f"
"https://github.com/user-attachments/assets/8621d3e8-ecb8-4860-afb2-5594f2ee17db"
)
echo "Verifying demo video URLs..."
for url in "${urls[@]}"; do
status=$(curl -o /dev/null -s -w "%{http_code}" "$url")
if [[ "$status" == "200" ]]; then
echo "✅ $url is reachable (HTTP 200)"
else
echo "⚠️ $url returned HTTP $status"
fi
doneLength of output: 1503
🏁 Script executed:
#!/bin/bash
# Follow redirects and check final HTTP status and redirect targets for each demo asset URL.
urls=(
"https://github.com/user-attachments/assets/8b2a0050-5ec4-424c-b417-8269971503f0"
"https://github.com/user-attachments/assets/d80b0776-4eb9-4e…
"https://github.com/user-attachments/assets/8621d3e8-ecb8-4860-afb2-5594f2ee17db"
)
echo "Following redirects and verifying final HTTP status:"
for url in "${urls[@]}"; do
echo
echo "URL: $url"
# Show the redirect chain
curl -I -s "$url" | awk '/^HTTP/{print; next} /^Location:/{print}'
# Follow redirects and print final status
final_status=$(curl -s -o /dev/null -w "%{http_code}" -L "$url")
echo "Final HTTP status after redirects: $final_status"
doneLength of output: 226
Ensure demo video URLs are publicly accessible
The current links under github.com/user-attachments all return HTTP 302 (redirect to sign-in), which means they aren’t publicly reachable. Please replace them with direct, publicly accessible URLs (for example, host the video files in the repo’s assets/ directory or on a public CDN).
Affected file:
- community/README.ru.md (lines 138–146)
🤖 Prompt for AI Agents (early access)
In community/README.ru.md around lines 138 to 146, the demo video URLs currently point to github.com/user-attachments and return HTTP 302 redirects to sign-in pages, making them inaccessible publicly. Replace these URLs with direct links to publicly accessible locations, such as hosting the video files in the repo's assets/ directory or on a public CDN, ensuring anyone can access the demo videos without authentication.
Description
DCO Affirmation
I affirm that all code in every commit of this pull request conforms to the terms of the Topoteretes Developer Certificate of Origin.