feat(javascript): add float16 support

[ayush00git]

Why?

Javascript package had the float16 support but during serialization it fallbacks to float32, much like the float16 was just a pseudo type which in real was a float32.

What does this PR do?

1. Adds float16(v: number | string) method to the BinaryWriterBuilder class

  float16(v: number | string) {
    return `${this.holder}.float16(${v})`
  }

2. Updated the float16 accessor in the Codegen Regsitery

CodegenRegistry.register(TypeId.FLOAT16,
  buildNumberSerializer(
    (builder, accessor) => builder.writer.float16(accessor),
    builder => builder.reader.float16()
  )
);

3. Updated the writer/number.ts and added float16 test suites in test/number.test.ts

Related Issue

• Closes [JavaScript] add float16 to javascript #3210

Does this PR introduce any user-facing change?

• Does this PR introduce any public API change?
• Does this PR introduce any binary protocol compatibility change?

Benchmark

N/A

[ayush00git]

Hey @chaokunyang @theweipeng
Please, take a look at the changes and suggest me what more changes are needed.

[chaokunyang]

Hi@ayush00git, thansk for opening up this PR. I think we need real float16 isntead of fallbakcing to float32

[ayush00git]

Hi@ayush00git, thansk for opening up this PR. I think we need real float16 isntead of fallbakcing to float32

Hey @chaokunyang
I this is the exact thing that i just fixed, previously it was falling back to float32, now its using the float16 accessor.
Or if is it something i am confused about, could you give me a exact reference of changes you want ?

[ayush00git]

Hey @chaokunyang @theweipeng
could you guide me on what exactly you need on this.

[chaokunyang]

@ayush00git could you add float16 array support in this pr?

[ayush00git]

@ayush00git could you add float16 array support in this pr?

Yaa sure, i'll add the arrays support

[ayush00git]

Hey @chaokunyang
The support for float16 array was already implemented in the PR #3197 by @theweipeng
here's the file - https://github.com/apache/fory/blame/main/javascript/packages/fory/lib/gen/typedArray.ts

CodegenRegistry.register(TypeId.FLOAT16_ARRAY, build(Type.float16()));

Still I've added separate tests for float16 array support, just as a proof of this.

[ayush00git]

Hey @chaokunyang
Could you let me know the changes, i'm working right now