chore: RenkoAtr BufferList removed (#1640)

Signed-off-by: Dave Skender <[email protected]>
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: DaveSkender <[email protected]>

Author: Copilot

docs/_indicators/Renko.md

@@ -38,19 +38,6 @@ You must have at least two periods of `quotes` to cover the warmup periods; howe
 
 **`EndType.HighLow`** - Brick change threshold measured from `High` and `Low` price
 
-## Chaining
-
-Results are based in `IQuote` and can be further used in any indicator.
-
-```csharp
-// example
-var results = quotes
-    .ToRenko(..)
-    .ToRsi(..);
-```
-
-This indicator must be generated from `quotes` and **cannot** be generated from results of another chain-enabled indicator or method.
-
 ## Response
 
 ```csharp
@@ -90,43 +77,23 @@ Each result record represents one Renko brick.
 
 See [Utilities and helpers]({{site.baseurl}}/utilities#utilities-for-indicator-results) for more information.
 
-## ATR Variant
-
-```csharp
-// C# usage syntax
-IReadOnlyList<RenkoResult> results =
-  quotes.ToRenkoAtr(atrPeriods, endType);
-```
-
-### Parameters for ATR
-
-**`atrPeriod`** _`int`_ - Number of lookback periods (`A`) for ATR evaluation.  Must be greater than 0.
-
-**`endType`** _`EndType`_ - See options below.  Default is `EndType.Close`
-
-#### Historical quotes requirements for ATR
-
-You must have at least `A+100` periods of `quotes`.
-
-`quotes` is a collection of generic `TQuote` historical price quotes.  It should have a consistent frequency (day, hour, minute, etc).  See [the Guide]({{site.baseurl}}/guide/#historical-quotes) for more information.
+## Chaining
 
-## Response for ATR
+Results are based in `IQuote` and can be further used in any indicator.
 
 ```csharp
-IReadOnlyList<RenkoResult>
+// example
+var results = quotes
+    .ToRenko(..)
+    .ToRsi(..);
 ```
 
-- This method returns a time series of all available indicator values for the `quotes` provided.
-- It does not return a single incremental indicator value.
-
-## Streaming and real-time usage for ATR variant
-
-**⚠️ Streaming not supported for ATR variant**: The `ToRenkoAtr()` method requires calculating ATR across the full dataset to determine the final brick size. Incremental streaming (StreamHub) or buffer (BufferList) implementations would require buffering all historical quotes and recalculating the entire Renko series on each new data point, which defeats the purpose of incremental processing.
-
-**Recommendation**: Use the Series implementation (`ToRenkoAtr()`) with periodic batch recalculation. For real-time scenarios with dynamic brick sizing, consider recalculating at appropriate intervals rather than on every tick. Alternatively, use the fixed brick size variant (`ToRenko()`) which fully supports streaming and buffering (see below).
+This indicator must be generated from `quotes` and **cannot** be generated from results of another chain-enabled indicator or method.
 
 ## Streaming
 
+**Fixed brick size only** - Streaming implementations are only available for fixed brick size Renko. The [ATR variant](#atr-variant) requires full dataset processing and does not support incremental streaming.
+
 Subscribe to a `QuoteHub` for streaming scenarios:
 
 ```csharp
@@ -141,8 +108,6 @@ foreach (Quote quote in quotes)  // simulating stream
 IReadOnlyList<RenkoResult> results = observer.Results;
 ```
 
-## Buffering
-
 Use a `BufferList` for incremental processing:
 
 ```csharp
@@ -156,8 +121,40 @@ foreach (Quote quote in quotes)  // simulating incremental data
 IReadOnlyList<RenkoResult> results = buffer;
 ```
 
-The buffering approach is ideal for growing datasets where quotes arrive incrementally over time.
+## ATR Variant
+
+```csharp
+// C# usage syntax
+IReadOnlyList<RenkoResult> results =
+  quotes.ToRenkoAtr(atrPeriods, endType);
+```
+
+### Parameters for ATR
+
+**`atrPeriod`** _`int`_ - Number of lookback periods (`A`) for ATR evaluation.  Must be greater than 0.
+
+**`endType`** _`EndType`_ - See options below.  Default is `EndType.Close`
+
+#### Historical quotes requirements for ATR
+
+You must have at least `A+100` periods of `quotes`.
+
+`quotes` is a collection of generic `TQuote` historical price quotes.  It should have a consistent frequency (day, hour, minute, etc).  See [the Guide]({{site.baseurl}}/guide/#historical-quotes) for more information.
+
+### Response for ATR
+
+```csharp
+IReadOnlyList<RenkoResult>
+```
+
+- This method returns a time series of all available indicator values for the `quotes` provided.
+- It does not return a single incremental indicator value.
+- See [RenkoResult](#renkoresult) above for detailed response structure.
 
-> &#128681; **Warning**: Unlike most indicators in this library, this indicator DOES NOT return the same number of elements as there are in the historical quotes.  Renko bricks are added to the results once the `brickSize` change is achieved.  For example, if it takes 3 days for a $2.50 price change to occur an entry is made on the third day while the first two are skipped.  If a period change occurs at multiples of `brickSize`, multiple bricks are drawn with the same `Timestamp`.  See [online documentation](https://www.investopedia.com/terms/r/renkochart.asp) for more information.
->
 > &#128073; **Repaint warning**: When using the `ToRenkoAtr()` variant, the last [Average True Range (ATR)]({{site.baseurl}}/indicators/Atr/#content) value is used to set `brickSize`.  Since the ATR changes over time, historical bricks will be repainted as new periods are added or updated in `quotes`.
+
+### Streaming limitations for ATR
+
+**ATR variant does not support streaming**: The `ToRenkoAtr()` method requires calculating ATR across the full dataset to determine the final brick size. Incremental streaming would require buffering all historical quotes and recalculating the entire Renko series on each new data point, which defeats the purpose of incremental processing.
+
+**Recommendation**: Use the Series implementation (`ToRenkoAtr()`) with periodic batch recalculation. For real-time scenarios, consider recalculating at appropriate intervals rather than on every tick.

specs/001-develop-streaming-indicators/tasks.md

@@ -162,7 +162,7 @@ Note on former deferrals: Some indicators were previously marked as deferred due
 - [x] **T052** Implement Pvo BufferList in `src/m-r/Pvo/Pvo.BufferList.cs` ✅
 - [x] **T053** Implement QuotePart BufferList in `src/_common/QuotePart/QuotePart.BufferList.cs` ✅
 - [x] **T054** Implement Renko BufferList in `src/m-r/Renko/Renko.BufferList.cs` ✅
-- [ ] **T055** Implement RenkoAtr BufferList in `src/m-r/RenkoAtr/RenkoAtr.BufferList.cs`
+- [ ] **T055** ~~Implement RenkoAtr BufferList~~ **NOT IMPLEMENTED** — ATR calculation requires full dataset to determine final brick size. Buffering all quotes and recalculating entire Renko series on each add would defeat the purpose of incremental processing. Series-only implementation maintained.
 - [x] **T056** Implement Roc BufferList in `src/m-r/Roc/Roc.BufferList.cs` ✅
 - [x] **T057** Implement RocWb BufferList in `src/m-r/RocWb/RocWb.BufferList.cs` ✅
 - [x] **T058** Implement RollingPivots BufferList in `src/m-r/RollingPivots/RollingPivots.BufferList.cs` ✅

src/m-r/RenkoAtr/RenkoAtr.Catalog.cs

@@ -29,7 +29,7 @@ public static partial class RenkoAtr
             .WithMethodName("ToRenkoAtr")
             .Build();
 
-    // BufferList and StreamHub not implemented - ATR calculation requires full dataset
-    // to determine final brick size. Incremental processing would require buffering all
-    // quotes and recalculating entire series on each add. Series-only implementation maintained.
+    // BufferList and StreamHub not implemented - would require buffering all quotes
+    // and recalculating entire Renko series on each add to maintain ATR accuracy.
+    // Series-only implementation maintained for correctness.
 }