[MXNET-12627] Fixed param coercion of clojure executor/forward

[paroda]

Description

Fixes an issue with clojure package (issue #12627)

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

• [x ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
• [x ] Changes are complete (i.e. I finished coding on this PR)
• [x ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

minor bugfix

[marcoabreu]

Hey, thanks for your contribution! Would you mind adding a test please?

[gigasquid]

Thanks @paroda for finding and fixing this bug. Looks good, just a couple small tweaks

• Could you please move the coerce-map->tuple-seq function out to the https://github.com/apache/incubator-mxnet/blob/master/contrib/clojure-package/src/org/apache/clojure_mxnet/util.clj namespace? This way it can be reused by other functions that might need that conversion.

• Please add a test for the conversion so that it help document it for others in https://github.com/apache/incubator-mxnet/blob/master/contrib/clojure-package/test/org/apache/clojure_mxnet/util_test.clj and then a quick test to exercise the forward with those extra args in https://github.com/apache/incubator-mxnet/blob/master/contrib/clojure-package/test/org/apache/clojure_mxnet/executor_test.clj

Thanks again for you help. It is much appreciated 😸

[paroda]

sure, i'll update it and add the tests.

[paroda]

oops! will reopen after the update

[marcoabreu]

Feel free to just leave the PR open

[stu1130]

Thanks for making the clojure binding better! @paroda
@mxnet-label-bot[pr-work-in-progress]

[paroda]

updated with tests

[gigasquid]

Looks great! Thanks for the improvement 😄

[gigasquid]

When the CI tests pass. It's good to merge - Thanks again!