Failed Object Conversion on Rhino.Compute/Hops Environment

Help
1

Hey guys,

I am having an issue while trying to use Hops/Rhino.Compute to receive data from Speckle.

The synchronous receiver works gracefully, and I am able to receive text, but something bad happens when I try to get geometry from my SpeckleObject.

image
image1494×412 28.7 KB

Here is my sender.

image
image712×427 28.9 KB

And my hops/rhino.compute receiver.

image
image1030×433 45 KB

Also, I have tried using the SpeckleToNative component inside the receiver definition and I get a similar error.

Any ideas of what might be causing it?

Oh, files:
receiver.gh (7.5 KB)
sender.gh (5.5 KB)

2 Likes
2

Hey @vwb welcome to our forum!

Does the receive operation work outside the Hops/Rhino.Compute context? If so, @Stam is the best one to help since he’s the author of the sync nodes!

3

Hey Matteo, everything works as a charm outside the hops/compute context.

Doesn’t feel like the problem is with the sync nodes though. For example: inside the hops context, if I use a sync-receiver, get my message from the SpeckleObject and use a sync-sender to commit, it works.

See this example:

image
image1430×534 62.7 KB

And it works:

image
image986×361 20.2 KB

The exception happens when I attemp to get Rhino.Geometry from my SpeckleObject.

4

Back at the trenches. I figured out the ToNative/ToSpeckle and some of the Speckle components are working asynchronously, which in my experience is hardly digested by Rhino.Compute.

So I went there to write my little C# components to make conversions.

private void RunScript(object B, ref object D, ref object tol)
  {

    // Grab kit and converter.
    var kit = Speckle.Core.Kits.KitManager.GetDefaultKit();
    var converter = kit.LoadConverter(Speckle.Core.Kits.Applications.Rhino);

    // Provide context document to my converter.
    var doc = RhinoDoc.CreateHeadless(null);
    converter.SetContextDocument(doc);

    if (converter.CanConvertToNative(B as Speckle.Core.Models.Base))
    {
      D = converter.ConvertToNative(B as Speckle.Core.Models.Base);
      _converted = true;
    }

  }

bool _converted;

public override void AfterRunScript()
  {
    if (!_converted)
    {
      Component.Params.Output[0].ClearData();
    }
  }

So far so good. I was able to convert Points, Polylines, Meshes. But when I tried a Brep… :boom: Oops…

An exception occurred while processing request
System.NullReferenceException: Object reference not set to an instance of an object.
at Objects.Converter.RhinoGh.ConverterRhinoGh.BrepToNative(Brep brep)
at Objects.Converter.RhinoGh.ConverterRhinoGh.ConvertToNative(Base object)
at Script_Instance.RunScript(Object B, Object& D, Object& tol) in c:\Users\Victor\AppData\Local\Temp\z5oimfvg.0.cs:line 68

So I went on this method on the source and found this line:

 public RH.Brep BrepToNative(Brep brep)
    {
       var tol = RhinoDoc.ActiveDoc.ModelAbsoluteTolerance;
//...

RhinoDoc.ActiveDoc is null on the Rhino.Compute/Hops context. I was expecting the Converter to use the doc I passed on converter.SetContextDocument(doc).

Does it make any sense?

conversion-test.gh (9.6 KB)

1 Like
5

Hey @vwb ,
Yes it makes sense! Clearly an oversight from our side, I’m afraid there might be other conversions that need fixing.

I’ve reported it here: Rhino/GH converter should use the ContextDocument and not the ActiveDoc · Issue #745 · specklesystems/speckle-sharp · GitHub

Feel free to make a PR, otherwise we’ll tackle it in the next sprint, and sorry for the slow reply!

1 Like
6

Thanks for the help @teocomi :love_you_gesture:.

I opened the PR here: Rhino/GH Converter should use ContextDocument first, then fallback to ActiveDoc #749

Hope it helps!

1 Like