• Login
sketchucation logo sketchucation
  • Login
πŸ€‘ SketchPlus 1.3 | 44 Tools for $15 until June 20th Buy Now

Call for attention - context handlers and validation procs

Scheduled Pinned Locked Moved Developers' Forum
24 Posts 10 Posters 33.3k Views
Loading More Posts
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • T Offline
    thomthom
    last edited by 18 Oct 2010, 18:43

    I hope the various scripters here can take a moment and look into this.

    While working on a large siteplan where I had ~200-300K edges I noticed that when I had selected everything and right-clicked SU would freeze up for several seconds before poping up the window. Often the SU window would flicker as it returned to a responsive state and cause the right click menu to disappear.

    I tried with all plugins disabled and found that there was then no lag at all.

    It occurred to me that some plugins might be trying to process the whole selection in the context handler. Checking my own plugin I found Selection Toys to do so - which of course is a naught thing to do. I modified it so it'd only inspect the selection if the selection was less than 1000 entities. When the selection is over 1000 it makes the assumption that the selection can contain anything and display all menus.

    That cut down the lag a little, but there's still many seconds lag. So there is more plugins. Initial scan of my plugins I found no more, so now I ask the rest of you to review your context menu handlers.

    And I expect one should be just as careful when it comes to validation procs. (I'm a wee bit unsure to exactly when these triggers - as toolbar buttons also can have these.)

    Summary:
    Don't process the whole selection in context handlers or validation procs. Place size limitations or remove the validation all together.

    Test the right click menu and it's performance on large models with large selections when adding context handlers.


    Debug Script for Benchmarking

    Ensure this loads before all other plugins.
    Open the Ruby Console, when right-clicking performance data will be outputted.

    To list all attached handlers:
    TT.list_context_handlers

    Thomas Thomassen β€” SketchUp Monkey & Coding addict
    List of my plugins and link to the CookieWare fund

    1 Reply Last reply Reply Quote 0
    • T Offline
      tbd
      last edited by 19 Oct 2010, 02:38

      maybe we should create a checklist for releasing ruby scripts, something like http://lite.launchlist.net

      SketchUp Ruby Consultant | Podium 1.x developer
      http://plugins.ro

      1 Reply Last reply Reply Quote 0
      • T Offline
        thomthom
        last edited by 19 Oct 2010, 06:27

        I think that would be a good idea. πŸ‘

        Thomas Thomassen β€” SketchUp Monkey & Coding addict
        List of my plugins and link to the CookieWare fund

        1 Reply Last reply Reply Quote 0
        • A Offline
          AdamB
          last edited by 19 Oct 2010, 17:08

          I'm half finished with a plugin called "Snitch" which collates the time each plugin's Observers take so we can start pointing fingers.

          Guidelines are fine, but witchhunts / naming and shaming has much better results. πŸ˜„

          Developer of LightUp Click for website

          1 Reply Last reply Reply Quote 0
          • T Offline
            thomthom
            last edited by 19 Oct 2010, 17:40

            How do you manage to track other observers?
            Can it be translated into validation procs and context handlers?

            Thomas Thomassen β€” SketchUp Monkey & Coding addict
            List of my plugins and link to the CookieWare fund

            1 Reply Last reply Reply Quote 0
            • C Offline
              Chris Fullmer
              last edited by 19 Oct 2010, 19:46

              @adamb said:

              Guidelines are fine, but witchhunts / naming and shaming has much better results. πŸ˜„

              Woohoo!, let the games begin. I have a feeling I'm gonna score big!!!! (I hope its good to score big πŸ’š )

              Lately you've been tan, suspicious for the winter.
              All my Plugins I've written

              1 Reply Last reply Reply Quote 0
              • T Offline
                thomthom
                last edited by 19 Oct 2010, 19:50

                http://static.tvtropes.org/pmwiki/pub/images/torches_and_pitchforks.jpg

                Thomas Thomassen β€” SketchUp Monkey & Coding addict
                List of my plugins and link to the CookieWare fund

                1 Reply Last reply Reply Quote 0
                • T Offline
                  thomthom
                  last edited by 19 Oct 2010, 20:03

                  uh... both maybe πŸ˜„

                  [flash=640,385:2c4nksvr]http://www.youtube.com/v/zrzMhU_4m-g?fs=1&hl=en_US[/flash:2c4nksvr]

                  Thomas Thomassen β€” SketchUp Monkey & Coding addict
                  List of my plugins and link to the CookieWare fund

                  1 Reply Last reply Reply Quote 0
                  • H Offline
                    honoluludesktop
                    last edited by 19 Oct 2010, 20:05

                    Are the witches hunting, or are we hunting witches?-)

                    Don't know what the following has to do with this topic, but it seemed like the thing to do:-)

                    http://www.google.com/images?q=tbn;lnizWqr0oVMvhM;;www.wyldeart.com/Galleries/Expressionist/Edvard_Munch/Images/TheScream1Up.jpg&t=1&h=196&w=148&usg=__v8o0X36oacv_3bHuA0QFQ6a4ucw=

                    1 Reply Last reply Reply Quote 0
                    • H Offline
                      honoluludesktop
                      last edited by 19 Oct 2010, 20:14

                      That's rich! The opening scene strikes a cord, it's me writing an app:-)

                      temp00.jpg

                      1 Reply Last reply Reply Quote 0
                      • T Offline
                        thomthom
                        last edited by 28 Oct 2010, 12:39

                        I made a little sniffer script that will log the scripts that creates context handlers. Lets one easier inspect which scripts are adding context handlers.

                        I found xLine to be processing the whole selection. I'll be looking further into what scripts have inefficient context handlers. At the moment it takes many seconds for me when I right click a large set of selected entities.

                        (Removed - See first post)

                        Type TT.list_context_handlers to get a list of attached context handlers in the Ruby Console.

                        Thomas Thomassen β€” SketchUp Monkey & Coding addict
                        List of my plugins and link to the CookieWare fund

                        1 Reply Last reply Reply Quote 0
                        • T Offline
                          thomthom
                          last edited by 29 Oct 2010, 10:24

                          I just updated the first post with the debug script - it now benchmarks the handlers, outputting the time they take to process to the Ruby Console.

                          Thomas Thomassen β€” SketchUp Monkey & Coding addict
                          List of my plugins and link to the CookieWare fund

                          1 Reply Last reply Reply Quote 0
                          • T Offline
                            thomthom
                            last edited by 29 Oct 2010, 10:28

                            Sample output for a selection of about 100K entities:

                            ### Context Handler ### 2D#loader.rb:181:inactivate' [...] 2D#.rb:44
                            0.0 seconds

                            Context Handler

                            bezier.rb:501 [...] bezier.rb:501
                            0.006 seconds

                            Context Handler

                            parametric.rb:320 [...] (eval):2696
                            0.0 seconds

                            Context Handler

                            (eval):103:in `load' [...] pb_loader.rb:9
                            0.0 seconds

                            Context Handler

                            tt_bitmap2mesh.rb:29 [...] tt_proxy_loader.rb:28
                            0.0 seconds

                            Context Handler

                            tt_image_opacity.rb:30 [...] tt_proxy_loader.rb:28
                            0.0 seconds

                            Context Handler

                            ui_manager.rb:261:in `build_ui' [...] tt_selection_toys.rb:240
                            0.004 seconds

                            Context Handler

                            xLine.rb:51 [...] xLine.rb:51
                            0.0 seconds

                            Context Handler

                            Lib6Plugin.rb:656:in `build_config_handlers' [...] ZLoader__FredoScale.rb:24
                            0.0 seconds

                            Context Handler

                            Lib6Plugin.rb:656:in `build_config_handlers' [...] ZLoader__OnSurface.rb:26
                            0.489 seconds

                            Context Handler

                            (eval):299:in `initialize' [...] dynamiccomponents.rb:38
                            0.0 seconds

                            Context Handler

                            webtextures_loader.rb:886 [...] webtextures.rb:22
                            0.0 seconds

                            Context Handler

                            bezierspline_main.rb:2413:in load_all_menus' [...] extensions.rb:36:in load'
                            5.657 seconds`

                            (Note I had modified xLine and Selection Toys prior to this test)

                            Thomas Thomassen β€” SketchUp Monkey & Coding addict
                            List of my plugins and link to the CookieWare fund

                            1 Reply Last reply Reply Quote 0
                            • fredo6F Offline
                              fredo6
                              last edited by 7 Nov 2010, 07:16

                              @thomthom said:

                              Sample output for a selection of about 100K entities:

                              Context Handler

                              bezierspline_main.rb:2413:in load_all_menus' [...] extensions.rb:36:in load'
                              5.657 seconds[/ruby]

                              I fixed the one for BezierSpline, with an upgrade.
                              see http://forums.sketchucation.com/viewtopic.php?f=323&t=13563&p=100509#p100509

                              Fredo

                              1 Reply Last reply Reply Quote 0
                              • T Offline
                                thomthom
                                last edited by 7 Nov 2010, 10:07

                                πŸ‘

                                Thomas Thomassen β€” SketchUp Monkey & Coding addict
                                List of my plugins and link to the CookieWare fund

                                1 Reply Last reply Reply Quote 0
                                • P Offline
                                  Pout
                                  last edited by 10 Nov 2010, 14:02

                                  Reminds me of something simular I seem to have in big models when rightclicking a material.
                                  The rightclick window just doesn't pop-up

                                  1 Reply Last reply Reply Quote 0
                                  • T Offline
                                    thomthom
                                    last edited by 10 Nov 2010, 14:31

                                    @pout said:

                                    Reminds me of something simular I seem to have in big models when rightclicking a material.
                                    The rightclick window just doesn't pop-up

                                    Yea - I've noticed that too. Very annoying. But that menu isn't accessible to ruby plugins. So I think that one is on Google's shoulders.

                                    Thomas Thomassen β€” SketchUp Monkey & Coding addict
                                    List of my plugins and link to the CookieWare fund

                                    1 Reply Last reply Reply Quote 0
                                    • J Offline
                                      Jim
                                      last edited by 23 Jan 2011, 16:25

                                      @unknownuser said:

                                      @thomthom said:

                                      Sample output for a selection of about 100K entities:

                                      Context Handler

                                      bezierspline_main.rb:2413:in load_all_menus' [...] extensions.rb:36:in load'
                                      5.657 seconds[/ruby]

                                      I fixed the one for BezierSpline, with an upgrade.
                                      see http://forums.sketchucation.com/viewtopic.php?f=323&t=13563&p=100509#p100509

                                      Fredo

                                      Bezierspline_main is still sort-of slow for me - it takes 8 seconds for the context menu to pop-up for 75,000 selected entities. (I admit to having a sub-standard pc.)

                                      Hi

                                      1 Reply Last reply Reply Quote 0
                                      • fredo6F Offline
                                        fredo6
                                        last edited by 23 Jan 2011, 21:19

                                        @jim said:

                                        Bezierspline_main is still sort-of slow for me - it takes 8 seconds for the context menu to pop-up for 75,000 selected entities. (I admit to having a sub-standard pc.)

                                        Jim,

                                        I made a change in the script.
                                        see main post on BezierSpline

                                        Is is better with this release

                                        Thanks

                                        Fredo

                                        1 Reply Last reply Reply Quote 0
                                        • T Offline
                                          thomthom
                                          last edited by 23 Jan 2011, 21:24

                                          πŸ‘

                                          Thomas Thomassen β€” SketchUp Monkey & Coding addict
                                          List of my plugins and link to the CookieWare fund

                                          1 Reply Last reply Reply Quote 0
                                          • 1
                                          • 2
                                          • 1 / 2
                                          • First post
                                            Last post
                                          Buy SketchPlus
                                          Buy SUbD
                                          Buy WrapR
                                          Buy eBook
                                          Buy Modelur
                                          Buy Vertex Tools
                                          Buy SketchCuisine
                                          Buy FormFonts

                                          Advertisement