Hi @maciejka
Please can I work on this?

Hi @maciejka Please can I work on this?

HMMMM

@maciejka let work and optimize this part of the code,

here is how i will approach,

• Change Buffer Type: Update the codec module to use Array as the default buffer type for the Encode trait instead of ByteArray.
  -Update Encoding Logic: Adjust the encoding logic to serialize data directly into Array. This may involve modifying the Encode trait implementation to work natively with Array.

• Test and Benchmark: After implementing the change, thoroughly test the encoding and hashing process to ensure data integrity and then benchmark it to verify the performance improvement.

Hey @maciejka Can I take this up, as I have worked with encode trait before

Interested too. Exercise I've already done and am familiar with Encode

hey @maciejka In the code, dest.append_word_rev(val, 1 or 2 or 3); we're appending val as either a 1-byte, 2-byte, or 3-byte value. Given that we’re trying to push these length values into an array, which uses 4 bytes (32 bits) per element, I’m wondering can we do we pad the remaining bytes with zeros when pushing into the array? or is there have anyother way .
Also do you have any guidance on how to convert a u32 number into its little-endian form and append in the array ?

hey @maciejka In the code, dest.append_word_rev(val, 1 or 2 or 3); we're appending val as either a 1-byte, 2-byte, or 3-byte value. Given that we’re trying to push these length values into an array, which uses 4 bytes (32 bits) per element, I’m wondering can we do we pad the remaining bytes with zeros when pushing into the array? or is there have anyother way . Also do you have any guidance on how to convert a u32 number into its little-endian form and append in the array ?

You can have a look at how it is implemented ByteArray: https://github.com/starkware-libs/cairo/blob/d390b0904fd7cc7d5a0b39b61a504a25483740de/corelib/src/byte_array.cairo#L36

There is a pending_word for data that does not take full word (in ByteArray case it is felt252, in our case it is u32).

For conversion see: https://github.com/starkware-libs/cairo/blob/d390b0904fd7cc7d5a0b39b61a504a25483740de/corelib/src/byte_array.cairo#L227