Steven Abreu
Steven Abreu
Still having the same issue when running the tutorial in a Jupyter notebook. Works fine if I run it as a Python file. Any progress on fixing this?
Thanks for the quick reply @bamsumit. The licensing should not be a problem, I will check with the others. Would MIT or BSD-3 be okay? The code is still work-in-progress,...
Thanks for all your help so far @bamsumit. I've been struggling to get the weights and parameters to match between an original lava-dl-generated .net file and the .net file that...
Yeah, the batch dimension shouldn't be there
To re-open this, do we agree that all NIR graphs should have the batch dimension removed? Or is this something that should go into the newly added metadata field? We...
Hi there, thanks a lot for catching this! Could you add a simple test input that gives you this failure? And is the `shape` variable a `list`, or a single...
I made a PR to fix this, see #101, could you please confirm that it works for you now? Then I will merge it. Thanks again for your help!
Good idea, I've added a flag as NIRGraph parameter (`ensure_validity`). I've also put a `strict` flag into the read and write functions for NIRGraphs. This is turned on by default...
Yeah, supporting string padding is not really necessary (it's a higher level of abstraction, and an IR is not the place for that), plus it adds more complexity. It also...
I see your point, but it might also be useful to leave it to keep NIR explicit and expressive. Any other opinions? (no rush on this, we can resolve this...