From bf68ccf0469161ab8479e7949d4adb286b8d8eef Mon Sep 17 00:00:00 2001
From: IrfanUddinAhmad <irfanahmad@arbisoft.com>
Date: Mon, 29 Jan 2024 15:28:27 +0500
Subject: [PATCH] fix: Refactored logging and default probability

---
 CHANGELOG.rst             |  9 +++++++++
 skill_tagging/__init__.py |  2 +-
 skill_tagging/pipeline.py | 10 +---------
 3 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
index 0cf4b8e..6be6765 100644
--- a/CHANGELOG.rst
+++ b/CHANGELOG.rst
@@ -16,6 +16,15 @@ Unreleased
 
 *
 
+[0.1.6] - 2024-01-29
+************************************************
+
+Changed
+=======
+
+* Refactored logging and default run probability
+
+
 [0.1.5] - 2024-01-25
 ************************************************
 
diff --git a/skill_tagging/__init__.py b/skill_tagging/__init__.py
index 3b19593..147bfdc 100644
--- a/skill_tagging/__init__.py
+++ b/skill_tagging/__init__.py
@@ -2,7 +2,7 @@
 Django app plugin for fetching and verifying tags for xblock skills.
 """
 
-__version__ = '0.1.5'
+__version__ = '0.1.6'
 
 # pylint: disable=invalid-name
 default_app_config = 'skill_tagging.apps.SkillTaggingConfig'
diff --git a/skill_tagging/pipeline.py b/skill_tagging/pipeline.py
index 5652fa0..90dd27d 100644
--- a/skill_tagging/pipeline.py
+++ b/skill_tagging/pipeline.py
@@ -10,7 +10,7 @@
 from openedx_filters import PipelineStep
 
 logger = logging.getLogger(__name__)
-DEFAULT_PROBABILITY = 0.5
+DEFAULT_PROBABILITY = 0.03
 
 
 def resource_string(path):
@@ -84,10 +84,6 @@ def run_filter(self, block, fragment, context, view):  # pylint: disable=argumen
 
         # Check whether we need to run this filter and only call the API.
         if not self.should_run_filter():
-            logger.info(
-                "[Xblock-Skill-Tagging] Filter aborted for vertical block. Block-ID: %s",
-                str(block.scope_ids.usage_id.block_id)
-            )
             return {"block": block, "fragment": fragment, "context": context, "view": view}
         skills = self.fetch_related_skills(block)
         if not skills:
@@ -128,10 +124,6 @@ def run_filter(self, block, context):  # pylint: disable=arguments-differ
         """Pipeline Step implementing the Filter"""
         usage_id = block.scope_ids.usage_id
         if usage_id.block_type != "video" or not self.should_run_filter():
-            logger.info(
-                "[Xblock-Skill-Tagging] Filter aborted for block. Block-ID: %s",
-                str(block.scope_ids.usage_id.block_id)
-            )
             # avoid fetching skills for other xblocks
             return {"block": block, "context": context}
         skills = self.fetch_related_skills(block)