asyml / asyml/forte

Allow `is_span` and `is_audio_span` to process entries in primitive form

Open
#901 0 comments 0 reactions 0 assignees View on GitHub
data_efficiency
Dominant language
Python
Stars
253
Forks
59
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**
Currently, the `in_span` and `in_audio_span` methods only handle entries of type `Union[int, Entry]`. We want to allow these methods to also handle entries that are stored in a dictionary form according to the interface suggested in #900.

**Describe the solution you'd like**
There are two ways in which this problem can be approached.

1. Modify each conditional branch of the methods to handle entries of the type `Dict[str, Any]`
2. Create new methods `in_span_raw` and `in_audio_span_raw` which would specifically be focused on handling entries of type `Dict[str,Any]`.

Contributor guide

Open the contributing guide

Research direction

Start by locating the `in_span` and `in_audio_span` methods and read issue #900 for the proposed dictionary interface. Determine whether the existing methods or new raw variants should accept `Dict[str, Any]`; done means primitive-form entries work for both span types without breaking existing `int` and `Entry` inputs.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.