Feat: Add wrap and neighbor access methods for grid logic

[HossamSaberr]

This PR implements the boundary and adjacency logic.

• Wrapping: Added atColumnWrap:atRowWrap: and atColumnWrap:atRowWrap:put: to allow seamless boundary wrapping using 1-based modulo arithmetic.

• Neighbors: Added neighborsAtColumn:atRow: to safely fetch up to 8 surrounding valid elements without out-of-bounds errors.

• Testing: Added tests in CTArray2DTest covering normal access, edge wrapping and a full 8-neighbor.

• Fixes About wrap and neighbors #17

[jordanmontt]

This PR has conflicts now :/

[HossamSaberr]

Resolved!

[jordanmontt]

@Ducasse could you review this, please?

[virenvarma007]

@HossamSaberr Nice work on this PR overall.
The wrapping API (atColumnWrap:atRowWrap: and ...put:) is clean, and the added tests for wrapping reads/writes plus neighbor behavior are solid.

@Ducasse one small follow-up point: issue #17 mentions neighbors “with or without wrapping at edges.” Right now we have non-wrapping neighbors and wrapping for direct access, but not a wrapped-neighbors variant yet. Also, neighborsAtColumn:atRow: does not currently enforce in-bounds origin coordinates, so clarifying expected behavior there would help. What do you think?

[HossamSaberr]

Thanks for the review, @virenvarma007!

Spot on with the wrapped variant, I'll add neighborsAtColumnWrap:atRowWrap: to handle that case

the out-of-bounds origin for neighborsAtColumn:atRow:: I think throwing an error here is much better

I'll push the updates for both of these, along with the tests

[HossamSaberr]

I've pushed the updates to address the points you raised @virenvarma007
The CI and Tests all Green, ready for another look

[virenvarma007]

This looks really good to me. Although we should wait for @Ducasse ’s take before we call it fully done, but either way, @HossamSaberr thank you for the effort you put into this.

[jordanmontt]

The PR looks ready to merge! I would just add a method comment explaining what the method is doing to the methods:

• atColumnWrap:atRowWrap:put:
• atColumnWrap:atRowWrap:
• neighborsAtColumn:atRow:
• neighborsAtColumnWrap:atRowWrap:

Because they can get tricky to understand at first sight

[HossamSaberr]

Fair enough! i would do it and push the commit in couple of hours

[HossamSaberr]

done! pushed the commit adding the requested method comments

Everything is good , CI and tests are green