Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
KTO: Minor fix and documentation update (#574)
## Summary <!--- This is a required section; please describe the main purpose of this proposed code change. ---> 1. Modifies the logic to correctly handle the case where `ref_average_log_prob_chunk` is set to None. 2. Corrects the KTO loss documentation. <!--- ## Details This is an optional section; is there anything specific that reviewers should be aware of? ---> ## Testing Done <!--- This is a required section; please describe how this change was tested. ---> <!-- Replace BLANK with your device type. For example, A100-80G-PCIe Complete the following tasks before sending your PR, and replace `[ ]` with `[x]` to indicate you have done them. --> - Hardware Type: A100-SXM4-80GB - [x] run `make test` to ensure correctness - [ ] run `make checkstyle` to ensure code style - [x] run `make test-convergence` to ensure convergence
- Loading branch information