-
Notifications
You must be signed in to change notification settings - Fork 24
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
Getter Docstrings #1185
base: main
Are you sure you want to change the base?
Getter Docstrings #1185
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1185 +/- ##
==========================================
- Coverage 90.69% 89.58% -1.11%
==========================================
Files 129 129
Lines 8189 8357 +168
==========================================
+ Hits 7427 7487 +60
- Misses 762 870 +108
Flags with carried forward coverage won't be shown. Click here to find out more.
|
After the meeting today, I think this kind of task would be a really good benchmark for AI agents / LLMs / prompts. Some quantifiable questions:
With that in mind, it might be worthwhile to branch off from neuroconv with a dedicated ai_docstrings_benchmark branch before merging these changes. |
Added Returns section to all getter docstrings.
Fixes #1097
Cline (Claude 3.5 Sonnet) helped. It was so close to just getting it right after the first couple attempts, but then took a lot of coaching to get those last few...