Add offset in dynamic_bitset

[Alex-PLACET]

No description provided.

[Copilot]

Pull Request Overview

This PR adds offset support to the dynamic_bitset infrastructure to handle array slicing correctly. The changes enable bitset operations to work with a logical offset into the underlying buffer without needing to adjust the buffer pointer.

• Added offset parameter to dynamic_bitset constructors and propagated through the class hierarchy
• Modified bit access operations to account for the offset when indexing into the buffer
• Updated the arrow array schema proxy to use the new offset-aware constructor

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
arrow_array_schema_proxy.cpp	Updated to pass offset parameter when creating dynamic_bitset_view
dynamic_bitset_view.hpp	Added offset parameter to constructors and forwarded to base class
dynamic_bitset_base.hpp	Implemented offset storage and applied offset to bit access operations

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 3 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The offset() method declaration is missing implementation. Add the implementation for this method to return m_offset.

Suggested change

	[[nodiscard]] size_t offset() const noexcept;
	[[nodiscard]] size_t offset() const noexcept
	{
	return m_offset;
	}

[Copilot]

Adding offset to the size for block count calculation is incorrect. The block count should be based on the actual storage needed for the bits, not the logical size plus offset. This could cause buffer overruns when the offset pushes beyond allocated memory.

Suggested change

	const size_type new_block_count = compute_block_count(n + m_offset);
	const size_type new_block_count = compute_block_count(n);