Excel Connector hangs when loading some commits

jhdempsey86 commented 1 month ago

Prerequisites

[x] I read the contribution guidelines
[x] I checked the documentation and found no answer.
[x] I checked existing issues and found no similar issue.
[x] I checked the community forum for related discussions and found no answer.
[x] I'm reporting the issue to the correct repository (see also speckle-server, speckle-sharp, specklepy, speckle-docs, and others)

What package are you referring to?

Speckle Connector for Excel

Describe the bug

When trying to get the data from the following commit, the Excel connector shows the loading circles, but the circles keep spinning indefinitely, and the UI hangs and becomes unresponsive. https://speckle.xyz/streams/846ea197e7/commits/7a32e5f0e5

This commit was produced using the code changes supplied in #3578 in the speckle-sharp repo

Commits made from the Civil 3D connector which don't have the above changes all seem to work fine

To Reproduce

Load up the Speckle Connector in Excel Select the "Jonathan's First Project" project It will try and load the latest commit At this point the loading circle will start to display, and will never go away, and the UI becomes unresponsive

Expected behaviour

We should be able to see the model in the viewing section, and download the data

Screenshots

System Info

Desktop (please complete the following information):

Windows 11 Excel Office 365

Failure Logs

The following is from the debugger which is available within Excel:

The list of API calls made doesn't show any errors

No excel log files could be found at C:\Users\<>\AppData\Roaming\Speckle\Logs Should there be any?

Additional context

N/A

Proposed Solution (if any)

Its unclear what's happening as I can see when I select a commit that some data is downloaded, (I'm guessing the graphical model) but then the model is never displayed and you see the loading circle indefinitely. I don't think its related to the data or metadata because I am unable to even click the receive button to even start downloading that data.

Optional: Affected Projects

https://speckle.xyz/streams/846ea197e7/commits/7a32e5f0e5

teocomi commented 1 month ago

Thanks for the report @jhdempsey86 we'll look into this! Our team is a bit stretched, so it might take a bit, in the meantime, you could try with data sent from Revit - that's a more tested workflow.

PS Any specific reason not to use the new web app? https://app.speckle.systems/

jhdempsey86 commented 1 month ago

Usually, the connector works fine with data from Civil 3D. It works on every other project and model I have ever pushed into Speckle.

Its just one particular version of a model that has the problem. We edited the Civil 3D connector with @clairekuang to push a lot more data about a Civil 3D corridor object, and that is what is contained in the linked project version. So there is something we have added into Speckle using that updated connector that the Excel connector doesn't like. But we have no idea what it might be.

specklesystems / speckle-excel