Spaces:
Running
Running
Commit
·
c4a9ea8
1
Parent(s):
2e33030
Update CLAUDE.md with reasoning trace parsing fix documentation
Browse files- Document the reasoning trace parsing fix added 2025-08-07
- Explain the problem, solution, and benefits
- Update existing reasoning trace section to note well-formed requirement
- Add technical details about the fix
CLAUDE.md
CHANGED
@@ -52,6 +52,8 @@ The application now supports displaying reasoning traces from models like NuMark
|
|
52 |
**Implementation Details:**
|
53 |
- New `reasoning-parser.js` module handles detection and parsing of reasoning traces
|
54 |
- Supports multiple reasoning formats (`<think>`, `<thinking>`, `<reasoning>` tags)
|
|
|
|
|
55 |
- Formats numbered steps from reasoning content for structured display
|
56 |
- Caches parsed reasoning to avoid reprocessing
|
57 |
- Exports include optional reasoning trace content
|
@@ -329,4 +331,29 @@ The application now displays model processing information when available:
|
|
329 |
**Implementation Notes:**
|
330 |
- The model info panel only appears when `inference_info` column exists
|
331 |
- Supports datasets processed with UV scripts via HF Jobs
|
332 |
-
- Gracefully handles datasets without model metadata
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
52 |
**Implementation Details:**
|
53 |
- New `reasoning-parser.js` module handles detection and parsing of reasoning traces
|
54 |
- Supports multiple reasoning formats (`<think>`, `<thinking>`, `<reasoning>` tags)
|
55 |
+
- **Important:** Only well-formed traces with both opening AND closing tags are parsed
|
56 |
+
- Malformed traces (missing closing tags) are displayed as plain text
|
57 |
- Formats numbered steps from reasoning content for structured display
|
58 |
- Caches parsed reasoning to avoid reprocessing
|
59 |
- Exports include optional reasoning trace content
|
|
|
331 |
**Implementation Notes:**
|
332 |
- The model info panel only appears when `inference_info` column exists
|
333 |
- Supports datasets processed with UV scripts via HF Jobs
|
334 |
+
- Gracefully handles datasets without model metadata
|
335 |
+
|
336 |
+
### Reasoning Trace Parsing Fix (Added 2025-08-07)
|
337 |
+
|
338 |
+
Fixed an issue where reasoning traces with incomplete or malformed XML tags would cause parsing errors:
|
339 |
+
|
340 |
+
**Problem:**
|
341 |
+
- Some model outputs contained opening `<think>` tags without closing `</think>` tags
|
342 |
+
- This appeared to be truncated or malformed model output
|
343 |
+
- The parser would attempt to parse these incomplete traces, causing confusion
|
344 |
+
|
345 |
+
**Solution:**
|
346 |
+
- Updated `detectReasoningTrace()` to require BOTH opening and closing tags
|
347 |
+
- Added console warnings when incomplete traces are detected
|
348 |
+
- Malformed traces are now displayed as plain text instead of being parsed
|
349 |
+
|
350 |
+
**Benefits:**
|
351 |
+
- Cleaner handling of incomplete model outputs
|
352 |
+
- No confusing partial reasoning panels for malformed content
|
353 |
+
- Maintains full functionality for well-formed reasoning traces
|
354 |
+
- Helpful console warnings for debugging
|
355 |
+
|
356 |
+
**Technical Details:**
|
357 |
+
- File: `js/reasoning-parser.js`
|
358 |
+
- Only traces with complete XML tags (`<think>...</think>`, `<thinking>...</thinking>`, etc.) are parsed
|
359 |
+
- Incomplete traces log: "Incomplete reasoning trace detected - missing closing tags"
|