Add Ellipses as valid type for remote execution
Closed this issue · 2 comments
JadenFiotto-Kaufman commented
Add Ellipses as valid type for remote execution
francescortu commented
Hi, I'm working on it and francescortu@77f1005 should work. However since the server doesn't have the EllipsisType, it currently raises an error ( I guess it isn't able to compile the EllipsisType since it doesn't know what is it). Is it reasonable?
JadenFiotto-Kaufman commented
@francescortu Looks perfect! I would use Ellipsis
instead of type(...)
. And yeah NDIF will have to be built with the version of nnsight that includes this change. If you wanted to you could build a dev version of NDIF with the nnsight version pointed at your nnsight version: https://github.com/ndif-team/ndif