Remove LLMChain and refactor
Open
@HavenDV is already working on this.
Since Feb 2, 2024.
enhancement
- Dominant language
- C#
- Stars
- 1.1k
- Forks
- 141
- Avg merge
- 2d 18m
- Merged PRs (30d)
- 2
Description
Peter James — Yesterday at 10:40 AM
Something else interesting / confusing is that there are two LLMChain classes in relatively close proximity (LangChain.Chains.HelperChains vs. LangChain.Chains.LLM). They both end up implementing IChain, but it seems one of them is for the pipeline syntax? Also, the two classes use different capitalization LLMChain vs LlmChain.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.