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

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 😸

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

oops! will reopen after the update

Feel free to just leave the PR open

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

updated with tests

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