Skip to content

Conversation

@mariaelissat
Copy link
Contributor

No description provided.

@gitronald gitronald self-assigned this Mar 7, 2025
@gitronald gitronald requested a review from Copilot March 7, 2025 18:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Overview

This PR updates the header text mappings in the Spanish version of the header_text classifier. The key changes include adding Spanish translations for various header types, adjusting header value lists, and updating mappings for keys such as "directions", "knowledge", and "videos".

Reviewed Changes

File Description
WebSearcher/classifiers/header_text.py Updated header mappings to include Spanish translations and additional header texts

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

WebSearcher/classifiers/header_text.py:109

  • The 'videos' mapping contains duplicate entries for "Videos"; consider removing the redundancy unless it is intentional.
        "videos": ["Videos", "Videos"]

fix missing commas

Co-authored-by: Copilot <[email protected]>
@gitronald
Copy link
Owner

Thanks Maria!

@gitronald gitronald merged commit 10a60a5 into gitronald:dev Mar 7, 2025
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.

2 participants