Please.Stop.Introducing.Breaking.Changes

Otherwise the developers, who are still trying to do something with Sentis, will go back to ORT.
It would be good to have someone in the team, who can control the usability and backward compatibility of the product, its components, API, etc.

Yes sorry for that @roumenf we apologize.
Now that we are no longer a “preview” package we will be much more conservative with breaking changes.

Thank you, but please tell me how to work around these issues now:

  • FunctionalTensor.FromTensor() is changed from public to internal. Why was this needed, and what would be the correct way to initialize a FunctionalTensor from a tensor or constant now?
  • The previously public properties of TextureTensorData like shape, bufferAsTexture, widthShift, widthMask, dimAxisDiv4, dimAxis, blockAxis are changed to internal too. Again, why was this needed at all, and how is one supposed to get their values now?
1 Like
  • Functional.Constant consolidated all constant creation.
  • TextureTensorData, that is a oversight they should have stayed public. we’ll do a patch to address it

Chiming in here with a follow-up question regarding Functional.Constant if you don’t mind. Is there a reason the method takes an int[] size parameter as input as opposed to a TensorShape?
Trying to do the right thing™ re allocations which seems more difficult with the Functional API.

It’s ok. That code isn’t GC free at all atm.
It’s mainly meant to be a editor time thing where you’d bake (serialize) the model for runtime use afterwards

Any ETA for the patch fixing the TextureTensorData accessibility issues?

1 Like

@alexandreribard_unity
Given unity/sentis-blaze-face · Hugging Face is outdated due to these breaking changes, I’ve been trying to convert the example code to sentis 2.0.

Where I get stuck is in the compilation of the functionalgraph.
Previously I could just use a lambda expression, and pass data around, grabbing InputDef.FromModel(model)[0]

While the changes for things like ReadbackAndClone and Worker are clear…
This isn’t really well documented in the upgrade-guide and I’m a bit stuck.