Skip to content

[Repo Assist] Fix delayed series display: show key range instead of (Suppressed) — closes #65#662

Merged
dsyme merged 2 commits intomasterfrom
repo-assist/fix-delayed-series-display-65-c72cd778f9d7e370
Mar 19, 2026
Merged

[Repo Assist] Fix delayed series display: show key range instead of (Suppressed) — closes #65#662
dsyme merged 2 commits intomasterfrom
repo-assist/fix-delayed-series-display-65-c72cd778f9d7e370

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Repo Assist — addressing a long-standing usability issue with delayed/lazy series display.

Summary

When a delayed (lazy) series was printed (in FSI, ToString(), or Format()), it showed the opaque string (Suppressed), which gave users no useful information about the series they had just created.

Before:

series [ (Suppressed)]
```
**After:**
```
series [ (Delayed series [2000-01-01 .. 2023-12-31])]

Root Cause

DelayedVector.SuppressPrinting returns true to prevent data loading during printing. The printing code in Series.fs branched on this flag but simply showed "(Suppressed)". However, the key range is already available in DelayedIndex.KeyRange which returns (source.RangeMin, source.RangeMax) — both set at construction time, with no data loading required.

Fix

In Series.ToString() and Series.FormatStrings(), when vector.SuppressPrinting is true, access series.KeyRange to get the [lo .. hi] bounds and format a descriptive message. A try/with guard preserves backward compatibility for any edge case.

Changes

  • src/Deedle/Series.fs: two one-line-to-four-line improvements in ToString() and FormatStrings()

Test Status

✅ Build succeeded
✅ All 670 tests pass (dotnet test tests/Deedle.Tests/Deedle.Tests.fsproj -c Release)

The existing test suite does not include a test for delayed-series printing. Adding one would require the DelayedSeries test infra; maintainers may wish to add a unit test for this if they consider it important.

Generated by Repo Assist ·

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@30f2254f2a7a944da1224df45d181a3f8faefd0d

When a delayed (lazy) series is printed, show the key range
[lo .. hi] instead of the opaque '(Suppressed)' message.

This implements the suggestion in #65. The key range is available
from DelayedIndex.KeyRange = (source.RangeMin, source.RangeMax)
without triggering data loading.

Before: series [ (Suppressed)]
After:  series [ (Delayed series [2000-01-01 .. 2023-12-31])]

A try/with guard keeps backward compatibility for any edge case
where the key range is not accessible.

Closes #65

Co-authored-by: Copilot <[email protected]>
@dsyme dsyme marked this pull request as ready for review March 19, 2026 01:53
@dsyme dsyme merged commit e1fe36a into master Mar 19, 2026
2 checks passed
@dsyme dsyme deleted the repo-assist/fix-delayed-series-display-65-c72cd778f9d7e370 branch March 19, 2026 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant