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)