Currently there is no option to create streams or add stream by ID as mentioned in DUI Makeover issue. Implement this option with 2.11

Description

Inspired by Evolvelab’s Helix.

Currently, when a model is received in Sketchup, the interface freezes for a while, and then the model appears. During this building activity, the entire interface is blocked. Instead, it should run the building activity in the background without blocking the UI/freezing, similar to Beautiful Builds implemented in Revit.

When the video recordings below are watched, what is meant to be explained will be understood more clearly.

Steps to reproduce

1. Send any model from Revit to Speckleverse
2. Try to receive this model via Sketchup connector.
3. The interface will be blocked for a while during the receiving/construction process.

Streams/Commits Used

https://speckle.xyz/streams/480108d3de

Files Used

Screenshots

The current way of constructing geometry

Expected Behaviour

This is kinda blocked atm as I'm not sure how to go about it yet. Basically, since we smoosh a bunch of meshes together on to_speckle, the texture coords and vertices don't match when recreating meshes on to_native.

Will be added below gems to increase code quality and to achieve easier maintanence

• RuboCop is a real shover tool for code quality
  • rubocop-minitest
  • rubocop-rake
  • rubocop-sketchup
• Rake is the make-like program implemented in Ruby to run tasks and dependencies on CI.
• RubyCritic static analysis through code base

After that Rakefile will be defined to run them on CI before merging to main.

There are some helper tools an methods while developing on SketchUp.

• jf_rubyLoader to reload files has changed when SketchUp instance is already opened. This will be only available when developing, so that's why also we should split into two piece speckle_connector.rb file, first itself, second speckle_connector_loader.rb
• Helper methods to check running states on SketchUp, those are just aliases for long methods in SketchUp, like Sketchup.active_model.selection.to_a

Optional: Affected Projects

Need to check changing loader will effect server connection? I do not think so

Ignore dimension objects. They cause breaking down.

Toolbar button to create a stream, send selection, and open in browser all with one click

Converters are the most complex classes on the repository currently and lacking documentation.
After implementing RuboCop to merging workflow, some classes under the Converters still disabled (waiting for refactor).

Speckle Objects will be created for conversions

• Geometry
  • Length
  • Point
  • Vector
  • Plane
  • BoundingBox
  • Plane
  • Mesh
• Other
  • Transform
  • RenderMaterial
  • BlockDefinition
  • BlockInstance
• Primitive
  • Interval

Expected vs. Actual Behavior

Meshes that are not in any group/component takes significantly longer. Especially geometries that are a bit complex takes forever to send.

Reproduction Steps & System Config (win, osx, web, etc.)

Download the file from below link, send it to Speckleverse🟦.

https://drive.google.com/file/d/17jwqMrPap31us8onEcpsAZwmu3KT_-L4/view?usp=sharing

Proposed Solution (if any)

Group meshes/faces by material or layer (maybe this can be in Advanced Settings too).

Optional: Affected Projects

💄DUI Makeover for Sketchup Connector

OK SO a couple things to improve with mesh conversions to_native

• components should only be created from meshes received from BIM applications
  • i think this just means anything in a displayValue becomes a component and any other mesh remains but i am sure there's an edge case there somewhere
  • this will require a bit of a complex/delicate restructure in the converter so be careful 🙃
• consider complex raw meshes leaving sketchup (curvy surfaces that are actually a bunch of tris but in the ui are somehow presenting as one object
• how do we add the edges onto re-received components? I'm adding the edges to the definition but for some reason it's still not showing up. do i need to do some intersection to re-"attach" them to the faces?

for blocks, block instances, lines, meshes

Gotta update the blocks going out of SketchUp to use the new Transform type. When receiving, we can check if it's an old or new transform style and grab the needed value as necessary (yay for dynamically typed languages!)

specklesystems/speckle-sharp#905

Implement all commands to achieve same functionality on 2.9

• send_selection -> convertedFromSketchup
• receive_objects -> finishedReceiveInSketchup
• reload_accounts -> loadAccounts
• init_local_accounts -> loadAccounts
• load_saved_streams -> setSavedStreams
• save_stream -> setSavedStreams
• remove_stream -> setSavedStreams
• notify_connected -> convertedFromSketchup + oneClickSend

Expected vs. Actual Behavior

Receiving block instances is a bit unreliable and strange. It seems to affect nested blocks in particular. There are a few different failure scenarios that can happen

• the block definition fails at some stage within the recursive creation, so a partially empty definition is created
• something catastrophic happens during conversion and sketchup crashes (!?)
• the block definition appears, but the instances do not

Reproduction Steps & System Config (win, osx, web, etc.)

Re-receive the hostel sample model into Sketchup

Description

• faces of quad meshes from blender dont get build in sketchup. only edges.

  

  https://latest.speckle.dev/streams/28dd9ad7ba/commits/16bc12df71
  commit used☝

  https://drive.google.com/file/d/1lLqvbLQPbNbJQLD5y5ewP8c1aU12ggps/view?usp=sharing
  file used☝

Description

Include Advanced Settings option in the DUI Makeover.

• Dark/Light Mode

Send Options

• Combines faces by materials
• Include entity attributes

Receive Options

• Merge co-planar faces on receive

Optional: Affected Projects

Issue #52

The Sketchup side of this issue specklesystems/speckle-sharp#783

Enable method check for definition as below,

geometry = if definition.entities[0].is_a?(Sketchup::Edge) | definition.entities[0].is_a?(Sketchup::Face)
                       group_mesh_to_speckle(definition, units, definitions)
                     else
                       definition.entities.map { |entity| convert.call(entity) }
                     end

speckle objects have a lot of additional info that is lost when receiving in sketchup

we should have an advanced setting in the ui that allows users to toggle on/off the receiving of these additional props as component attributes. this way, they can also be re-attached when sending out

since component attributes are only key:value and stringifying a dictionary to dump into a value wouldn't really be useful, we could either

1. ignore nested props as a first pass
2. pull the props all up to the top level and re-nest them on send (eg { "thing": { "name": "izzy"} } gets attached to the attrs as thing.name => "izzy"

Description

To be more consistent across connectors, we should have do DUI makeover for Sketchup connector.

• Create New Stream

  

• Add Stream by ID

  

• Saving streams (on send and receive)

  

• More options on send(Everything - Layers - Views - Selection)

  

• Receive Options (Update - Create - Ignore)

  

• Auto-Receive

  

• Advanced Settings (issue # here)

  

• Preview on receive (not priority but good to have)

  

remove matomo and implement new stuff!

Expected vs. Actual Behavior

Meshes from SketchUp need to be constructed on a face by face basis. Currently, the faces are iterated through and all the vertices and faces are pushed to a list to create the Speckle Mesh which creates duplicates. This is fine for most applications, but Revit is a very unhappy boi

Proposed Solution (if any)

Create a middle step between getting mesh info an constructing the Speckle mesh to avoid duplicated vertices and re-map the poly indices for the faces

This bug reported via community under the post. Main problem is grouping meshes according to their materials when they are under the group or component. Instead group meshes, theoratically we should send them separately because when we receive back they will be handled as separate objects. And also realized that currently we do not send orphan edges (edge that does not involve any face) in the group.

PS: Performance should be checked.

Let's try release this sprint!

• Finalize alpha for wider announcement
• Create basic set of docs / tutorials

Description

• Trying to receive something i sent from sketchup in Revit. I get this error👇:

  name
  Parameter name: name contains illegal characters.
  A group has been changed outside group edit mode. The change is being allowed because there is only one instance of the type.

  https://latest.speckle.dev/streams/28dd9ad7ba/commits/471b5644ec
  commit used☝.

  https://drive.google.com/file/d/1gO4vDJEdpDEswXI5UPMm3D1H5Iw2OzU-/view?usp=sharing
  file used☝

  

• No Materials in Revit when received from Sketchup.

  https://latest.speckle.dev/streams/28dd9ad7ba/commits/56673b9c0d
  commit used☝.

  

Tracking this one from the forum as some more users are reporting it now:

https://speckle.community/t/sketchup-authorisation-issues/2206

Summary

Blocks with both faces and edges currently exclude edges on send. This was initially for size and cosmetic reasons, but it has been noted on the forum that they are important for the editability of faces.

I guess to preserve full editability, all edges should be sent regardless of visibiliy.

This may require viewer tweaks to avoid busy / wareframe-y looking models.

Target User

requested on the forum
https://speckle.community/t/face-edges-missing-sketchup/2625/2

Notes & Challenges

• may need to experiment with merging lines into polys or grouping lines
• since we merge faces into meshes by material, this may cause difficulties on receive with edges and faces on perfectly lining up

Coordination

• possibly viewer coordination for possible block conditions (eg hide lines in mixed line and mesh definitions? though this would bork blocks that are intended to have polys and meshes so will need to think)

Docs & Content

• update docs with this behaviour and ping the forum with the update!

We should prevent to merge the coplanar faces that shouldn't be merged as below.
Originally there are 3 faces that created before SketchUp by intersecting 2 faces. Intersected area is created automatically by SketchUp's intersection algorithm. So when we try to get these 3 face from XYZ, we should not call merge_coplanar_faces method. To understand this one we should check existence of sketchup_attributes before merge.

• Launching with no account produces blank UI - should have error message as default
  
• If there's no connection, send cancels with no error message & receive hangs on "requesting" with no way to cancel the operation
  
• Arcs, circles, ellipses, and curves from RhinoTestStream are not receiving at all
  

save streams to files so they remain when relaunching the connector

Speckle Sketchup is currently in Alpha - let's get it to stable release!

• #4
• couple of render material bug fixes
• #25
• #23

This issue will serve to refactor codebase in the light of freshly implemented RuboCop and RubyCritic. Current state of RuboCop issues around 70-80. And RubyCritic as below,

Additionally, we have currently %0 coverage so there is no any unit or integrated tests.

Sketchup faces (Sketchup::Face) are tricky. When try to create face from mesh that comes from speckle.xyz, we should make sure that face edges are soften before or not. This flag important for faces in SketchUp, because it makes them editable. Because the way to maniputale faces is done through it's edges. This is how SketchUp faces works.

Following steps should be done to achieve

• Add sketchup_attributes object to all speckle objects. This will be the way transfer data.
• When sending faces to server as SpeckleObjects::Geometry::Mesh, send is_soften flag with sketchup_attributes
• When receiving meshes from server, check is_soften is set, if not assume that it's true else use value

By this way, we can keep faces same from SketchUp to SketchUp. For the meshes that comes from other connectors will be taken as soften edges for now, later we should re-consider it.

First of all, amazing work with Speckle. I love what you guys are doing.

I was installing speckle-sketchup on macos and found the following issue.

I have first built and installed the plugin following the instructions here.
When you launch the speckle connector you get the following error

Launching Speckle Connector from /Users/ruggiero/Library/Application Support/SketchUp 2021/SketchUp/Plugins/speckle_connector/html/index.html
Error: #<IOError: No Speckle Directory exists. Please read the guide to get Speckle set up on your machine: 
https://speckle.guide/user/manager.html>
/Users/ruggiero/Library/Application Support/SketchUp 2021/SketchUp/Plugins/speckle_connector/accounts.rb:51:in `_get_speckle_dir'
/Users/ruggiero/Library/Application Support/SketchUp 2021/SketchUp/Plugins/speckle_connector/accounts.rb:15:in `load_accounts'
/Users/ruggiero/Library/Application Support/SketchUp 2021/SketchUp/Plugins/speckle_connector/dialog.rb:84:in `init_local_accounts'
/Users/ruggiero/Library/Application Support/SketchUp 2021/SketchUp/Plugins/speckle_connector/dialog.rb:42:in `block in show_dialog'

This is caused by the way the platform is identified in accounts.rb. In Sketchup for macos RUBY_PLATFORM.downcase returns x86_64-darwin18 and this contains win.

According to this a better way to check for the platform is to use Sketchup.platform replacing the function with something like this:

    def self._get_speckle_dir
      # platform = RUBY_PLATFORM.downcase
      
      speckle_dir =
        if Sketchup.platform == :platform_win
          # win
          File.join(Dir.home, "AppData/Roaming/Speckle")
        elsif Sketchup.platform == :platform_osx
          # mac
          File.join(Dir.home, ".config", "Speckle")
        else
          # linux
          # Do you need this?
          File.expand_path("~/.local/share/Speckle")
        end
      return speckle_dir if Dir.exist?(speckle_dir)

      raise(IOError, "No Speckle Directory exists. Please read the guide to get Speckle set up on your machine: \nhttps://speckle.guide/user/manager.html")
    end

Tested on my mac and it works.

Hope this helps

some funny buggies found by @didimitrie tyty!

• guide points - they break shit!
• some deeply nested components are not getting surfaced - brutal model as an ex
• materials poss on wrong side of surfaces? check the market hall roof for debugging

• open in web
• add commit message
• switch branches
• create streams
• create branches
• search streams

somewhat align with desktopui 2

Add ported sketchup-sqlite3 repo as submodule and clear it from this repo.

Hi,
The installation of the connector is not possible if the name of the user, I used Jérôme, contains some characters (à, é, è, ç ...).
Here are the error messages.

Erreur de chargement du fichier C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector/main.rb
Error: #<LoadError: cannot load such file -- sqlite3>
C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require' C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require'
C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector/accounts.rb:8:in rescue in <top (required)>' C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector/accounts.rb:3:in <top (required)>'
C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require' C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require'
C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector/dialog.rb:5:in <top (required)>' C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require'
C:/Program Files/SketchUp/SketchUp 2022/Tools/RubyStdLib/rubygems/core_ext/kernel_require.rb:92:in require' C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector/main.rb:4:in <top (required)>'
C:/Program Files/SketchUp/SketchUp 2022/Tools/extensions.rb:197:in require' C:/Program Files/SketchUp/SketchUp 2022/Tools/extensions.rb:197:in load'
C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector.rb:39:in register_extension' C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector.rb:39:in module:SpeckleConnector'
C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector.rb:6:in <module:SpeckleSystems>' C:/Users/Jérôme/AppData/Roaming/SketchUp/SketchUp 2022/SketchUp/Plugins/speckle_connector.rb:5:in <top (required)>'

Regards.

SpeckleEntity is a class that was waiting for a ready playground to implement.

Now we have;

time to cook SpeckleEntity 👨‍🍳

Before that, traversing methods will be refactored to be aligned with SpeckleState and it's update procedure!

Layers in SketchUp seems as Tags. When we send/receive geometries from SketchUp to SketchUp, tags information get lost.

Expected vs. Actual Behavior

Objects from other connectors are now serialised with nulls. This throws an error in the object loader and causes receiving to fail in SketchUp. This needs to be fixed in ObjectLoader and updated here to fix and release a new ver.

related to specklesystems/speckle-server#519

Expected vs. Actual Behavior

Simple shapes comes as triangles. Lets merge them and get single faces.

Ugly Triangles📐

Beautiful Faces🔳

Workaround

Currently I use Cleanup plugin to clean those kind of edges/faces👇.

https://extensions.sketchup.com/extension/046175e5-a87a-4254-9329-1accc37a5e21/clean-up

Proposed Solution (if any)

Do a merge operation after receiving geometry.

edit from @izzylys: should add an option in #55 to toggle on/off this behaviour as it might not always be desirable (since you're turning triangulated meshes into ngon meshes)

Add repo specific contribution guide that explains newly added sketchup-sqlite3 submodule.

If it's your first time here - or you forgot about them - make sure you read the contribution guidelines, and then feel free to delete this line!

Expected vs. Actual Behavior

Describe the problem here.

Reproduction Steps & System Config (win, osx, web, etc.)

Let us know how we can reproduce this, and attach relevant files (if any).

Proposed Solution (if any)

Let us know what how you would solve this.

Optional: Affected Projects

Does this issue propagate to other dependencies or dependents? If so, list them here!

some things like the birdbath aren't showing up in the full send - investigate!

Currently the way of using sqlite3 in sketchup is installing gem on runtime which is not something sketchup suggests, and it might be the reason it is not working as desired always. See SketchupRequirements/GemInstall

# This is a current workaround.
begin
  require('sqlite3')
rescue LoadError
  # ty msp-greg! https://github.com/MSP-Greg/SUMisc/releases/tag/sqlite3-mingw-1
  Gem.install(File.join(File.dirname(File.expand_path(__FILE__)), '../../utils/sqlite3-1.4.2.mspgreg-x64-mingw32.gem'))
  require('sqlite3')
end

The way to go follow instructions of @thomthom on the repo to create sketchup compatible compiled .so and .bundle library to use as below.

require_relative 'ext/sqlite3'

Properties of the component instance and it's definition should be send/receive with correct data.

• Component Instance Name
• Component Definition Name
• Component Definition Description

Previously if we send empty component name, we are receiving them back with Speckle ids, it pollutes Outliner virtually.

add matomo metrics

A basic guide with a rundown of using the SketchUp connector with other connectors