Skip to content

[Design] [Form Recognizer | Table Indexer] Implement indexer on ExtractedTable to index into Cells array #9975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
annelo-msft opened this issue Feb 14, 2020 · 5 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer Cognitive Services
Milestone

Comments

@annelo-msft
Copy link
Member

annelo-msft commented Feb 14, 2020

Summary

Tables cells are available in Form Recognizer output in order to use them in application UI, rather than for data analysis. Given this, providing a table indexer that allows callers to index into a given cell in the table by row, col, is a more idiomatic approach than having an array of cells users must iterate through to find a given cell.

One question that arises is how to handle column span and row span.

We should understand prior approaches in BCL/.NET and design an approach for indexing into table cells consistent with these.

The resulting API will likely look like:

public class ExtractedTable {
    public ExtractedTableCell[] Cells { get; }
    ...

See:

Goals

  • Identify examples of table indexers elsewhere in BCL/.NET to understand precedents
  • Design an API for indexing into table cells consistent with other BCL/.NET types
  • Review design with feature crew and architects
  • Implement design
@annelo-msft annelo-msft added Client This issue points to a problem in the data-plane of the library. Cognitive Services FormRecognizer labels Feb 14, 2020
@annelo-msft
Copy link
Member Author

@ctstone @mayurid

@AlexGhiondea AlexGhiondea added this to the [2020] April milestone Mar 3, 2020
@AlexGhiondea
Copy link
Contributor

@annelo-msft could you please update the title?

@annelo-msft annelo-msft changed the title Form Recognizer Implement indexer on ExtractedTable to index into Cells array Mar 4, 2020
@kinelski kinelski self-assigned this Mar 20, 2020
@AlexGhiondea AlexGhiondea modified the milestones: [2020] April, [2020] May Apr 9, 2020
@annelo-msft annelo-msft assigned maririos and unassigned kinelski May 4, 2020
@kinelski kinelski modified the milestones: [2020] May, [2020] June May 6, 2020
@annelo-msft
Copy link
Member Author

#10537 to be done as part of this work.

@annelo-msft annelo-msft changed the title Implement indexer on ExtractedTable to index into Cells array [Form Recognizer | Table Indexer] Implement indexer on ExtractedTable to index into Cells array May 7, 2020
@annelo-msft annelo-msft changed the title [Form Recognizer | Table Indexer] Implement indexer on ExtractedTable to index into Cells array [Design] [Form Recognizer | Table Indexer] Implement indexer on ExtractedTable to index into Cells array May 7, 2020
@annelo-msft annelo-msft modified the milestones: [2020] June, [2020] July May 14, 2020
@annelo-msft annelo-msft added the blocking-release Blocks release label Jun 4, 2020
@maririos
Copy link
Member

We will like to get more information about the usage of this feature to help shape the design.
Moving to backlog

@maririos maririos removed the blocking-release Blocks release label Jul 10, 2020
@maririos maririos removed their assignment Jul 10, 2020
@pallavit
Copy link
Contributor

Closing this, and will reopen if there is a customer need for this feature.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer Cognitive Services
Projects
None yet
Development

No branches or pull requests

5 participants