[wasm] Add atan kernel

[chunnienc]

To see the logs from the Cloud Build CI, please join either our discussion or announcement mailing list.

---

This change is 

[mattsoulanille]

LGTM. Thanks for the contribution!

[pyu10055]

Thanks you for ramping up so quickly. a minor nitpick on the PR.

Reviewable status: complete! 1 of 1 approvals obtained (waiting on @chunnienc)

---

tfjs-backend-wasm/src/setup_test.ts line 405 at r1 (raw file):

  {include: 'reciprocal'},
  {include: 'isNaN'},
  {include: 'atan '},

please remove the trailing space, I think the description of atan tests only contains 'atan'.

[chunnienc]

Thanks you for ramping up so quickly. a minor nitpick on the PR.

Reviewable status: complete! 1 of 1 approvals obtained (waiting on @chunnienc)

tfjs-backend-wasm/src/setup_test.ts line 405 at r1 (raw file):

  {include: 'reciprocal'},
  {include: 'isNaN'},
  {include: 'atan '},

please remove the trailing space, I think the description of atan tests only contains 'atan'.

The test filter applies on the resolved/expanded test name e.g. this test's name for matching would be "atan basic", so it matches the filter with the trailing space.

Besides:

1. The current filter with trailing space could fail with my intended incorrect implementation, which means the atan tests are properly included.
2. The filter without trailing space failed since it includes other tests for 'atan2' and 'atanh'.

[pyu10055]

Reviewable status: complete! 2 of 1 approvals obtained (waiting on @chunnienc)

---

tfjs-backend-wasm/src/setup_test.ts line 405 at r1 (raw file):

Previously, chunnienc wrote…

The test filter applies on the resolved/expanded test name e.g. this test's name for matching would be "atan basic", so it matches the filter with the trailing space.

Besides:

1. The current filter with trailing space could fail with my intended incorrect implementation, which means the atan tests are properly included.
2. The filter without trailing space failed since it includes other tests for 'atan2' and 'atanh'.

awesome, thank you for the clarification, trailing space would be a good way to avoid other ops.