Skip to content

Timer report#995

Open
Christian-B wants to merge 22 commits into
masterfrom
timer_report
Open

Timer report#995
Christian-B wants to merge 22 commits into
masterfrom
timer_report

Conversation

@Christian-B

@Christian-B Christian-B commented Oct 21, 2022

Copy link
Copy Markdown
Member

This PR has.

  1. Fixes to the timings in ASB

    • add FecTimer.start_category(TimerCategory.GET_MACHINE, True) to actully record the machine on time
    • added the DATA_GENERATION category
  2. Adds a timer reports

    • run at the end of stop so it report over multiple resets (after a merge with Split provenance #990)
    • Report saved in timestamp directory as it covers all runs and all resets (or will after above merge)
    • Can also be run standalone over a previous run database (One created after the fixes above have gone in)

Jenkins tested with report always on but suggest turning it off by default. Especially as it can be easily done later.

Added tests to demonstrate stand alone

There is a microcircuit_model PR which runs and reports the timer reports in jenkins.
SpiNNakerManchester/microcircuit_model#21

@Christian-B Christian-B mentioned this pull request Oct 27, 2022
@dkfellows dkfellows added this to the 7.1.0 milestone Sep 27, 2023
@rowleya

rowleya commented Oct 5, 2023

Copy link
Copy Markdown
Member

I could be wrong but I think we have this already...

@rowleya rowleya closed this Oct 5, 2023
@Christian-B

Copy link
Copy Markdown
Member Author

What we have is a lot less than this

@Christian-B Christian-B reopened this Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants