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

Contrastive Reranker/Reward model #9171

Merged
merged 26 commits into from
Jul 10, 2024
Merged

Contrastive Reranker/Reward model #9171

merged 26 commits into from
Jul 10, 2024

Conversation

arendu
Copy link
Collaborator

@arendu arendu commented May 11, 2024

What does this PR do ?

enables training of a contrastive reranker model based on GPT architecture and PEFT. Used for reranking the results of an embedding model.

Collection: [NLP]

Changelog

  • Add specific line by line info of high level changes in this PR.

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this 

GitHub Actions CI

The Jenkins CI system has been replaced by GitHub Actions self-hosted runners.

The GitHub Actions CI will run automatically when the "Run CICD" label is added to the PR.
To re-run CI remove and add the label again.
To run CI on an untrusted fork, a NeMo user with write access must first click "Approve and run".

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

  • New Feature
  • Bugfix
  • Documentation

If you haven't finished some of the above items you can still open "Draft" PR.

Who can review?

Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.

Additional Information

  • Related to # (issue)

arendu added 5 commits May 8, 2024 23:47
Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
arendu added 2 commits May 11, 2024 18:37
Signed-off-by: arendu <[email protected]>
@github-actions github-actions bot added the CI label May 11, 2024
arendu added 2 commits May 12, 2024 05:15
Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
@arendu arendu added Run CICD and removed Run CICD labels May 12, 2024
@arendu arendu added Run CICD and removed Run CICD labels May 12, 2024
Signed-off-by: arendu <[email protected]>
@arendu arendu requested a review from cuichenx May 13, 2024 20:57
@arendu arendu added Run CICD and removed Run CICD labels May 13, 2024
arendu and others added 2 commits May 14, 2024 16:48
Copy link
Contributor

This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or update or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Jun 16, 2024
Copy link
Contributor

This PR was closed because it has been inactive for 7 days since being marked as stale.

@github-actions github-actions bot closed this Jun 24, 2024
@arendu arendu reopened this Jul 9, 2024
@arendu arendu added Run CICD and removed Run CICD labels Jul 9, 2024
@arendu arendu added Run CICD and removed Run CICD labels Jul 9, 2024
Copy link
Collaborator

@cuichenx cuichenx left a comment

Choose a reason for hiding this comment

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

LGTM. Looks like pipeline parallel to be verified and supported in a subsequent PR.

@arendu arendu merged commit 74e32c8 into main Jul 10, 2024
204 checks passed
@arendu arendu deleted the adithyare/embedding_reranking branch July 10, 2024 16:26
marcromeyn pushed a commit that referenced this pull request Jul 11, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
marcromeyn pushed a commit that referenced this pull request Jul 11, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
maanug-nv pushed a commit that referenced this pull request Jul 14, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
ertkonuk pushed a commit that referenced this pull request Jul 19, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
Signed-off-by: Tugrul Konuk <[email protected]>
tonyjie pushed a commit to tonyjie/NeMo that referenced this pull request Aug 6, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
Signed-off-by: tonyjie <[email protected]>
monica-sekoyan pushed a commit that referenced this pull request Oct 14, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
hainan-xv pushed a commit to hainan-xv/NeMo that referenced this pull request Nov 5, 2024
* wip contrastive reranker

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* wip

Signed-off-by: arendu <[email protected]>

* working reranker training and validation

Signed-off-by: arendu <[email protected]>

* default peft for reranker

Signed-off-by: arendu <[email protected]>

* validation time update

Signed-off-by: arendu <[email protected]>

* reranker test

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* reranker inference

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* updates

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* also can support rlhf style reward model loss

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* Apply isort and black reformatting

Signed-off-by: arendu <[email protected]>

* typo in cicd

Signed-off-by: arendu <[email protected]>

---------

Signed-off-by: arendu <[email protected]>
Signed-off-by: arendu <[email protected]>
Signed-off-by: Adi Renduchintala <[email protected]>
Co-authored-by: arendu <[email protected]>
Signed-off-by: Hainan Xu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants