Problem
As part of this PR effort and for the purpose of testing, we replicated the nmtwrapper logic within the rsmt2d repository. Subsequently, suggestions were put forward to relocate this code entirely to the rsmt2d repository. The purpose of this issue is to address the aforementioned suggestion, examine the implications of this change, and notify relevant stakeholders accordingly.
Originally suggested by @evan-forbes and @Wondertan in these comments #242 (comment) and #242 (comment)
the only thing the wrapper imports is the consts, so maybe it makes sense to accept those consts as args, then move the entire wrapper here
Or why not move the wrapper here completely then?
Problem
As part of this PR effort and for the purpose of testing, we replicated the nmtwrapper logic within the rsmt2d repository. Subsequently, suggestions were put forward to relocate this code entirely to the rsmt2d repository. The purpose of this issue is to address the aforementioned suggestion, examine the implications of this change, and notify relevant stakeholders accordingly.
Originally suggested by @evan-forbes and @Wondertan in these comments #242 (comment) and #242 (comment)