Skip to content

rename "disable_lcs" to "disable_longest_common_substring"#66

Closed
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D83155383
Closed

rename "disable_lcs" to "disable_longest_common_substring"#66
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D83155383

Conversation

@mgrange1998
Copy link
Contributor

Summary:

  • rename "disable_lcs" to "disable_longest_common_substring"
  • Add _compute_char_level_longest_common_subsequence_helper and _compute_word_level_longest_common_subsequence_helper options to TextInclusionAnalysisNode and input

Differential Revision: D83155383

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 24, 2025
@facebook-github-bot
Copy link
Contributor

@mgrange1998 has exported this pull request. If you are a Meta employee, you can view the originating diff in D83155383.

Summary:

- rename "disable_lcs" to "disable_longest_common_substring"
- Add _compute_char_level_longest_common_subsequence_helper and _compute_word_level_longest_common_subsequence_helper options to TextInclusionAnalysisNode and input

Differential Revision: D83155383
@facebook-github-bot
Copy link
Contributor

@mgrange1998 has exported this pull request. If you are a Meta employee, you can view the originating diff in D83155383.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 6899139.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants