Stop LLMFreeTextQuestions from generating choices-based content warnings#791
Stop LLMFreeTextQuestions from generating choices-based content warnings#791sjd210 wants to merge 1 commit into
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #791 +/- ##
==========================================
- Coverage 40.31% 40.31% -0.01%
==========================================
Files 547 547
Lines 23773 23773
Branches 2893 2893
==========================================
- Hits 9584 9583 -1
Misses 13288 13288
- Partials 901 902 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This feels like an acceptable stop-gap. But I would love to review the inheritance hierarchy here, and try to sort out the mess of If |
|
@jsharkey13 I've also realised (to add to the mess) that we absolutely ARE expecting these questions to have the |
isaacLLMFreeTextQuestions do not require achoicesfield, since they use amarkSchemeas its equivalent.We should therefore not be giving a warning when the
choicesfield is missing.