Release 1.1.0 by vishwanath1004 · Pull Request #97 · ELEVATE-Project/commons-backend

vishwanath1004 · 2026-10-13T04:56:57Z

No description provided.

Story #2022 : Primary and secondary theme mapping

coderabbitai · 2026-10-13T04:57:09Z

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

Source mode: Auto
Repository: Unavailable
Branch: Unavailable
File: No configuration file from this repository applied
Commit: Unavailable
Layers: defaults
Review profile: CHILL
Plan: Advanced
Run ID: 15c2c6b3-3064-4d27-987a-99094b6579d6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

rakeshSgr · 2026-10-13T05:32:12Z

 )
 from chatbot.utils.company_utils import get_company_queryset_for_user, get_user_company, get_user_profile
+from chatbot.utils.knowledge_service.theme_classification_service import (
+    ADMIN_THEME_WAIT_TIMEOUT_SECONDS, ThemeMessageLevel, enqueue_theme_classification, theme_outcome_message,


@vishwanath1004 why this required?

I have kept the max waiting time as 30 seconds for ADMIN_THEME_WAIT_TIMEOUT_SECONDS, after that the theme mapping will happens in background.

rakeshSgr

@vishwanath1004 Please check the comments

rakeshSgr · 2026-10-13T05:37:35Z

+        if not pending or response.status_code != 302:
+            return response
+        task_id, media_name = pending
+        outcome = wait_for_theme_classification(task_id, ADMIN_THEME_WAIT_TIMEOUT_SECONDS)


@vishwanath1004 how it works?

Previously, after saving the media, users were immediately redirected to the listing page. With the addition of theme mapping to the existing implementation, we’ve updated the flow so that navigation happens only after the theme mapping is complete. This ensures that users can see the mapped themes immediately on the listing page.

@vishwanath1004 we can remove this, it may take more, lets it happen in backend

rakeshSgr · 2026-10-13T05:40:15Z

+
+    def get_queryset(self, request):
+        return super().get_queryset(request).select_related('created_by').annotate(
+            primary_count=Count('primary_media', distinct=True),


@vishwanath1004 what is primary_media ?

It adds an number to the each theme, Its a count for each media how many resources are using this as primary theme.

rakeshSgr · 2026-10-13T07:56:44Z

+                'existing_secondary_themes': list(existing_names),
+                'needed': needed,
+                'document': self.document_context(document),
+                'feedback': feedback,


@vishwanath1004 are we storing the feedback ?

No we are not storing in db, feedback used if theme classification is failed any cases we are showing an message.

we can remove this

rakeshSgr · 2026-10-13T07:57:37Z

+
+logger = logging.getLogger('django')
+
+THEME_CLASSIFIER_ROUTE = '/theme_classifier'


@vishwanath1004 create a constant file theme, and move these constant there

rakeshSgr · 2026-10-13T07:57:58Z

+MEDIA_THEME_FIELDS = ['primary_theme', 'primary_theme_confidence', 'primary_theme_reasoning', 'needs_review']
+SEARCH_TEXT_MAX_CHARS = 2000
+THEME_CODE_HASH_LENGTH = 12
+NEAR_DUPLICATE_NEIGHBOUR_FLOOR = 0.3


@vishwanath1004 how it works ?

we are using that to check the duplicate names of secondary theme.

@vishwanath1004 what is 0.3 value ?

thats the minimum matching, if the existing secondary theme name is matching with new one then it will treats as duplicate one so it will not create the secondary theme it will reuse the existing one.

rakeshSgr · 2026-10-13T07:58:22Z

+SEARCH_TEXT_MAX_CHARS = 2000
+THEME_CODE_HASH_LENGTH = 12
+NEAR_DUPLICATE_NEIGHBOUR_FLOOR = 0.3
+NEAR_DUPLICATE_NEIGHBOUR_LIMIT = 10


@vishwanath1004 provide me the brief explanation for these keys

rakeshSgr · 2026-10-13T08:01:42Z

+
+def acquire_secondary_theme_lock() -> None:
+    with connection.cursor() as cursor:
+        cursor.execute('SELECT pg_advisory_xact_lock(%s)', [SECONDARY_THEME_LOCK_KEY])


@vishwanath1004 SECONDARY_THEME_LOCK_KEY ? how this works

This is the ID number of a database lock. The lock stops two classifications running at the same time from both creating the same new secondary theme.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

rakeshSgr · 2026-10-22T10:35:45Z

+    if not media:
+        logger.warning(f"{log_prefix} vector sync skipped: media not found")
+        return None
+    if media.company_bot and media.company_bot.provider == LLMProvider.OPENAI:


@vishwanath1004 we don't need to check the == LLMProvider.OPENAI , check in other places also

rakeshSgr · 2026-10-22T10:38:54Z

+THEME_WAIT_MAX_POLL_SECONDS = 3.0
+SECONDARY_THEME_MIN_WORDS = 2
+SECONDARY_THEME_MAX_WORDS = 4
+MAX_SECONDARY_THEME_COUNT = 10


move this to env

SECONDARY_THEME_COUNT this is the key to manage the number of secondary themes to assign each document hence this key i have moved to .env from the .bot file. And MAX_SECONDARY_THEME_COUNT is the upper limit for the SECONDARY_THEME_COUNT setting. The setting decides how many secondary themes each document can get (currently 2), and it can never be set above 10.

vishwanath1004 added 2 commits October 8, 2026 18:39

Story #2022 : Primary and secondary theme mapping

5263af7

Merge pull request #5 from vishwanath1004/2022

2124951

Story #2022 : Primary and secondary theme mapping

rakeshSgr reviewed Oct 13, 2026

View reviewed changes

Comment thread chatbot/admin/media_admin.py

rakeshSgr reviewed Oct 13, 2026

View reviewed changes

vishwanath1004 and others added 4 commits October 15, 2026 15:25

PR changes and export feature

7231ff1

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Added themes name into media table

49d521d

Secondary themes export chnages

d22a1fa

PR comments resolve

3300e27

rakeshSgr reviewed Oct 22, 2026

View reviewed changes

vishwanath1004 added 8 commits October 23, 2026 16:38

PR comments changes

e3b542f

PR comments resolve

253c55a

Release notes added

41620fe

Secondary theme mapping order change

42d4435

Folder structure changes and release doc updated

4fefc5d

Sample.env update

4c201cd

Merge conflicts resolve

2cd3754

Conflicts ressolve

453807e

rakeshSgr merged commit 491662d into ELEVATE-Project:release-1.2.0 Oct 26, 2026
1 check passed


		logger = logging.getLogger('django')

		THEME_CLASSIFIER_ROUTE = '/theme_classifier'

Conversation

vishwanath1004 commented Oct 13, 2026

Uh oh!

coderabbitai Bot commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

rakeshSgr left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai Bot commented Oct 13, 2026 •

edited

Loading