Open MaxPayne86 opened 1 year ago
Would it be possible to also add in the scripts that you used to generate the .json and .csv files?
Script is here. Problem is that currently a CoreAudioML submodule would be needed at least since:
import CoreAudioML.miscfuncs as miscfuncs
import CoreAudioML.networks as networks
import CoreAudioML.dataset as dataset
Let me know if it's okay. Refactoring the code to avoid these deps would probably divert from what we want to do.
RTNeural is intended to be compatible with C++14
Yup, committed by mistake, reverted.
Sure, having the CoreAudioML
imports in there is fine, although I guess it would be good to add a comment at the top of the script like # This script depends on the CoreAudioML library (https://github.com/link/to/the/repo)
.
@jatinchowdhury18 hello I would like to discuss this PR: I've seen that RTNeural introduces apis for importing pytorch models. On my end I've embedded engine validation inside the plugin. Btw if you think it's still cool to have it I can provide the missing files so that we can merge. Thanks a lot!
@MaxPayne86 Sure thing! My idea with some of the more recent commits was to add an interface that could be used to load PyTorch models directly from the state_dict
JSON representation. I think having the script that could be used to convert from a PyTorch-style JSON file to an RTNeural-style JSON file wouldn't directly conflict with that, but I do worry that it might be a little bit confusing having two different ways of solving the same problem. It's probably more of a documentation problem that I would need to solve on my end, but if you have some ideas that would be very helpful as well!
But anyway, for finishing up this PR, I think the steps would be:
main
transpose()
defined in multiple places)That would be fantastic, and thanks again for all your work on this!
Codecov Report
91.13% <0.00%> (-7.35%)
92.85% <0.00%> (-0.48%)
97.77% <0.00%> (-0.05%)
98.00% <0.00%> (-0.04%)
100.00% <0.00%> (ø)
100.00% <0.00%> (ø)
86.80% <0.00%> (ø)
100.00% <0.00%> (ø)
100.00% <0.00%> (ø)
100.00% <0.00%> (ø)
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more