-
Notifications
You must be signed in to change notification settings - Fork 34
Re-upgrade Claude to Sonnet 4.0 and instructed AI only to explain policies #2540
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
base: master
Are you sure you want to change the base?
Re-upgrade Claude to Sonnet 4.0 and instructed AI only to explain policies #2540
Conversation
… policies, no commentary or quotes
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2540 +/- ##
==========================================
+ Coverage 63.98% 71.15% +7.16%
==========================================
Files 55 54 -1
Lines 2113 1768 -345
Branches 264 217 -47
==========================================
- Hits 1352 1258 -94
+ Misses 708 459 -249
+ Partials 53 51 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Please use "Fixes," not "Resolves," otherwise the script doesn't work |
Fixed, thank you for the feedback. |
Please test the prompt with Claude chat a few times and drop the results in a Google doc |
This is the link to the file containing the tests: Thank you. |
Please show the full text result |
Please use specifically "Fixes #ISSUE", the format you have won't correctly close the original issue. Check out the docs for more info. If you set it up right, this panel on the right should indicate the issue being closed: |
Updated, thanks |
Updated, thanks. |
Also, please modify the changelog as we discussed post-meeting today to just include modifications to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
marking as requesting changes to receive sample results
Updated, thank you. |
We avoided the upgrade because Claude 4 was adding things like FDR quotes. Please create a document of some sample LLM summaries to ensure our prompting avoids these issues. |
Updated, this is the link to the doc: https://docs.google.com/document/d/1ECOwIq2mX0CjWyxAYDevdX6vvIFKsbyibaItvPdKT_c/edit?usp=sharing |
@SongmingFan123 if you could just remove the changes you made to |
This reverts commit e803c5a.
Fixed, thank you. |
Changes to the files have been made. Thank you. |
Fixes #2522
Fixes #2512