linksplatform / linksplatform/Data.Doublets

Repair Count logic

Open
#359 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
C#
Stars
14
Forks
5
PR merge metrics
No merged PRs in 30d

Description

https://github.com/linksplatform/Data.Doublets/blob/86197375ad05039154d0c706d070b5e2a2cfed5a/csharp/Platform.Data.Doublets/Memory/Split/Generic/SplitMemoryLinksBase.cs#L491-L503

```cs
if (AreEqual(source, any))
{
if (AreEqual(storedLinkValue.Target, target))
{
return GetOne();
}
}
if (AreEqual(target, any))
{
if (AreEqual(storedLinkValue.Source, source))
{
return GetOne();
}
}
return GetZero();
```

Note that this should also affect C++ code base. As @uselessgoddess told me, it is already fixed in Rust code base.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with csharp/Platform.Data.Doublets/Memory/Split/Generic/SplitMemoryLinksBase.cs at lines 491-503, then locate the corresponding C++ implementation. Compare both with the already-fixed Rust code to determine the intended Count behavior; done means the C# and C++ logic is consistent with Rust.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, csharp, rust
Domain
data
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.