Outdated FIXME comment in log10 matrix benchmark

[manit2004]

Location: docs/migration-guides/mathjs/benchmark/benchmark.log10.matrix.2d.size.js

Problem:
The createBenchmark1 function contained a FIXME comment:

// y = log10( x ); FIXME: support once we have `@stdlib/math/special/log10`

However, @stdlib/math/base/special/log10 already exists and is imported at the top of the same file (line 39). The FIXME was outdated - the functionality was available but simply not implemented in this benchmark function.

[stdlib-bot]

👋 Hi there! 👋

And thank you for opening your first issue! We will get back to you shortly. 🏃 💨

If you have any further questions while waiting for a response, please join our Zulip community to chat with project maintainers and other community members.

[kgryte]

@manit2004 You should read the comment again. We don't actually have @stdlib/math/special/log10, yet. Instead we only have the "base" scalar kernel. As such, the FIXME is still applicable and cannot be resolved at this time.

[manit2004]

@kgryte I apologize for the confusion and premature PR. You're absolutely right - I misunderstood the FIXME comment. My implementation using x.map(base) was just a workaround, not the proper solution the FIXME was anticipating.

I should have researched the difference between the /base/special/ and /special/ namespaces before making changes. The FIXME is still valid and should remain until the proper higher-level implementation exists. Thank you for the clarification and sorry for the noise.