Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

📊 Make profiler output a bit nicer. #2078

Merged

Conversation

thorstenhater
Copy link
Contributor

@thorstenhater thorstenhater commented Jan 12, 2023

  • Profiler output is now correctly aligned regardless of widths.
  • Python (and C++) can now actually set a limit on the nodes printed.

After (with a >=4% filter):

REGION                                  CALLS     WALL    THREAD        %
root                                        -    9.401    75.205    100.0
  advance                                   -    9.401    75.205    100.0
    integrate                               -    9.400    75.201    100.0
      current                               -    2.001    16.007     21.3
        zero                         29120728    0.518     4.142      5.5
        Granule_98_3D__all           24000600    0.459     3.675      4.9
        Granule_98_3D__soma_group    24000600    0.456     3.649      4.9
        Gran_CaPool_98               24000600    0.410     3.282      4.4
      state                                 -    1.861    14.888     19.8
        Granule_98_3D__soma_group    24000600    0.778     6.223      8.3
        Gran_CaPool_98               24000600    0.498     3.981      5.3
        Granule_98_3D__all           24000600    0.410     3.283      4.4
      events                         58241456    1.017     8.138     10.8
      voltage                        29120728    0.954     7.629     10.1
      ionupdate                      29120728    0.526     4.205      5.6
      threshold                      29120728    0.514     4.116      5.5
      samples                        29120728    0.513     4.103      5.5
      stimuli                        29120728    0.512     4.092      5.4
      post                           29120728    0.509     4.072      5.4
      stepsupdate                    29120728    0.500     4.003      5.3
      diffusion                      29120728    0.494     3.950      5.3

Before:

_p_ REGION                     CALLS      THREAD        WALL       %
_p_ root                           -      56.635       7.079   100.0
_p_   advance                      -      56.635       7.079   100.0
_p_     integrate                  -      56.633       7.079   100.0
_p_       current                  -      11.836       1.480    20.9
_p_         zero            29120728       3.133       0.392     5.5
_p_         Granule_98_3D__soma_group    24000600       2.816       0.352     5
_p_         Granule_98_3D__all    24000600       2.650       0.331     4.7
_p_         Gran_CaPool_98    24000600       2.291       0.286     4.0
_p_         MossyFiber_all     3840096       0.501       0.063     0.9
_p_         Golgi_98_3D_all     1280032       0.123       0.015     0.2
_p_         Golgi_98_3D_soma_group     1280032       0.120       0.015     0.2
_p_         Golgi_CaPool_CML     1280032       0.103       0.013     0.2
_p_         nernst           1280032       0.100       0.013     0.2
_p_       state                    -      11.343       1.418    20.0
_p_         Granule_98_3D__soma_group    24000600       4.918       0.615     8
_p_         Gran_CaPool_98    24000600       3.086       0.386     5.4
_p_         Granule_98_3D__all    24000600       2.288       0.286     4.0
_p_         MossyFiber_all     3840096       0.479       0.060     0.8
_p_         Golgi_98_3D_all     1280032       0.252       0.031     0.4
_p_         Golgi_98_3D_soma_group     1280032       0.180       0.023     0.3
_p_         Golgi_CaPool_CML     1280032       0.139       0.017     0.2
_p_       voltage           29120728       7.124       0.891    12.6
_p_       events            58241456       6.003       0.750    10.6
_p_       threshold         29120728       3.015       0.377     5.3
_p_       samples           29120728       2.976       0.372     5.3
_p_       ionupdate         29120728       2.961       0.370     5.2
_p_       stimuli           29120728       2.890       0.361     5.1
_p_       stepsupdate       29120728       2.868       0.358     5.1
_p_       post              29120728       2.820       0.353     5.0
_p_       diffusion         29120728       2.794       0.349     4.9
_p_       setup                  728       0.001       0.000     0.0
_p_     samplesetup              728       0.001       0.000     0.0
_p_     spikes                   728       0.000       0.000     0.0
_p_     eventsetup               728       0.000       0.000     0.0
_p_     sampledeliver            728       0.000       0.000     0.0
_p_   communication                -       0.000       0.000     0.0
_p_     enqueue                    -       0.000       0.000     0.0
_p_       sort                   728       0.000       0.000     0.0
_p_       merge                  728       0.000       0.000     0.0
_p_       setup                  728       0.000       0.000     0.0
_p_     walkspikes                 1       0.000       0.000     0.0
_p_     spikeio                    1       0.000       0.000     0.0
_p_     exchange                   -       0.000       0.000     0.0
_p_       gather                   1       0.000       0.000     0.0
_p_       sort                     1       0.000       0.000     0.0
_p_       gatherlocal              1       0.000       0.000     0.0

@thorstenhater thorstenhater merged commit eee16c7 into arbor-sim:master Jan 12, 2023
@thorstenhater thorstenhater deleted the ux/better-profiler-output branch January 12, 2023 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants