Skip to content

Commit dc100fd

Browse files
committed
refactor: remove Parallel.For from MemoryLeakTest
Convert Concurrent_MultipleThreads_ViewsChainCorrectly test to use sequential for loop instead of Parallel.For.
1 parent a93c1c7 commit dc100fd

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

test/NumSharp.UnitTest/Backends/NDArray.Base.MemoryLeakTest.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -320,14 +320,14 @@ private void CreateAndGetReferences(out WeakReference<NDArray> weakOriginal, out
320320
public void Concurrent_MultipleThreads_ViewsChainCorrectly()
321321
{
322322
var original = np.arange(1000);
323-
var views = new System.Collections.Concurrent.ConcurrentBag<NDArray>();
323+
var views = new List<NDArray>();
324324

325-
// Create views from multiple threads
326-
System.Threading.Tasks.Parallel.For(0, 100, i =>
325+
// Create views sequentially
326+
for (int i = 0; i < 100; i++)
327327
{
328328
var view = original[$"{i}:{i + 100}"];
329329
views.Add(view);
330-
});
330+
}
331331

332332
// All views should chain to original
333333
foreach (var view in views)

0 commit comments

Comments
 (0)