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.

Sign up for GitHub

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

Jump to bottom

Fix: QA, Default Config, and Transformer Model for QA #926

Merged
merged 64 commits into from
Dec 21, 2023
Merged

Fix: QA, Default Config, and Transformer Model for QA #926

merged 64 commits into from
Dec 21, 2023

Conversation

Prikshit7766
Copy link
Contributor

@Prikshit7766 Prikshit7766 commented Dec 13, 2023 •
edited

Description

  • Fix the QA Task
  • Standardize Config Parameters
  • Fix Transformer Model for QA
  • Removed langchain (PromptTemplate) dependency for Transformer Model
  • Sensitivity Test to New Language Models
  • Fix Evaluation in SycophancySample
  • Added Test for nlp task supported in langtest and Transformer Model for QA

Fixes

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Usage

Checklist:

  • I've added Google style docstrings to my code.
  • I've used pydantic for typing when/where necessary.
  • I have linted my code
  • I have added tests to cover my changes.

@Prikshit7766 Prikshit7766 changed the title Fix: QA , Default Config, and Transformer Model for LLM Fix: QA, Default Config, and Transformer Model for LLM Dec 13, 2023
@Prikshit7766 Prikshit7766 marked this pull request as draft December 13, 2023 19:01
@Prikshit7766 Prikshit7766 changed the title Fix: QA, Default Config, and Transformer Model for LLM Fix: QA, Default Config, and Transformer Model for QA Dec 13, 2023
This was linked to issues Dec 17, 2023
Enhancement: QA Data Structure, Standardize Config Parameters, and Transformer Model" #930
Closed
Expand Sensitivity Test to New Language Models #846
Closed
Prikshit7766 and others added 23 commits December 18, 2023 03:31
@Prikshit7766
updated Errors.E091
7f5fb16
@Prikshit7766
fix: lint
49c2917
@chakravarthik27
Update prompt template for commonsenseqa
5453387
@chakravarthik27
Update prompt for fiq & hellaswag in custom_types/helpers.py
25c297b
@Prikshit7766
restructure the code
12efed6
@Prikshit7766
fix:accuracy and fairness for qa
2698439
@Prikshit7766
resolve conflict
6a75030
@Prikshit7766
hf_utils.py: added login_with_token method
775d93a
@Prikshit7766
updatded error code
5972b14
@Prikshit7766
updated sample.py
1108f74
@Prikshit7766
test: added test_nlp_tasks.py
043b2cb
@Prikshit7766
test(test_huggingface_model.py): added TestTransformersModels
4620c6f
@Prikshit7766
updated test_nlp_task.py
8d95e49
@Prikshit7766
updated test_nlp_task.py
e8b2e34
@Prikshit7766
updated metrics
edd3e59
@Prikshit7766
updated sample.py
04bf5c0
@Prikshit7766
updated test_nlp_tasks.py
947d52c
@Prikshit7766
fix formatting
96d54a1
@Prikshit7766
test_nlp_tasks.py: updated pytest
63bc7e5
@Prikshit7766
updated test_nlp_tasks.py
5697d25
@Prikshit7766
sample.py: SycophancySample updated prompt_eval method
e95769a
@Prikshit7766
default_config: updated disinformation_config.yml
17f576e
@Prikshit7766
updated test_nlp_task.py
05a125f
@Prikshit7766 Prikshit7766 marked this pull request as ready for review December 21, 2023 13:48
@ArshaanNazir ArshaanNazir self-requested a review December 21, 2023 18:26
ArshaanNazir
ArshaanNazir approved these changes Dec 21, 2023
View reviewed changes
langtest/datahandler/datasource.py Outdated
@@ -55,6 +55,7 @@
"text": ["question"],
"context": ["context", "passage", "contract"],
"answer": ["answer", "answer_and_def_correct_predictions"],
"options":["opyions"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

fix typo

@ArshaanNazir ArshaanNazir merged commit c953954 into release/1.10.0 Dec 21, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ArshaanNazir ArshaanNazir ArshaanNazir approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Prikshit7766 @ArshaanNazir @chakravarthik27