Skip to content
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

Linter Cleanup #236

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Linter Cleanup #236

merged 2 commits into from
Sep 18, 2024

Conversation

schreiaj
Copy link
Collaborator

@schreiaj schreiaj commented Sep 17, 2024

Stack from ghstack (oldest at bottom):

Minor changes to clean up linter errors

[ghstack-poisoned]
schreiaj added a commit that referenced this pull request Sep 17, 2024
Minor changes to clean up linter errors

ghstack-source-id: 64fb7e87b95ad2d87c56fb8c52be26342d806fbe
Pull Request resolved: #236
[ghstack-poisoned]
Copy link
Collaborator

@arinkulshi-skylight arinkulshi-skylight left a comment

Choose a reason for hiding this comment

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

lint fixes LTGM.

@schreiaj schreiaj merged commit 04fcea2 into gh/schreiaj/1/base Sep 18, 2024
4 checks passed
@schreiaj schreiaj deleted the gh/schreiaj/1/head branch September 18, 2024 15:58
github-merge-queue bot pushed a commit that referenced this pull request Sep 18, 2024
* Linter Cleanup

Minor changes to clean up linter errors

ghstack-source-id: cb3de56cf9f18ccc20015ab3296c507a85fb66e6
Pull Request resolved: #236

* Highlight Low Confidence Extractions

Highlights low confidence extractions:
- Dynamic computation of error count
- Highlights low confidence rows

ghstack-source-id: 014d1a0a5067243cc28c930533762969a1fbb2cb
Pull Request resolved: #237
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