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

Energy stuff with reset #1252

Merged
merged 16 commits into from
Jan 3, 2025
Merged

Energy stuff with reset #1252

merged 16 commits into from
Jan 3, 2025

Conversation

Christian-B
Copy link
Member

@Christian-B Christian-B commented Dec 12, 2024

Makes changes to allow energy reports over multiple runs and reset

  1. Adds a mapping from n_run to n_reset in global_provenance.sqlite3
  • With extra views to access the provenance data by reset
  1. Compute Energy Gets the timer data just for this reset
  2. Virtual power cores come from placements and not the database
  3. ProvenanceReader.get_last_run_database_path removed as it as wrong! And not needed
  4. power_provenance now by run
  5. energy report per run (CUMULATIVE per reset!)

Fixes: #866

tested by:
SpiNNakerManchester/sPyNNaker#1513 (needs to be merged too)

This was referenced Dec 17, 2024
@Christian-B Christian-B merged commit fc9d271 into master Jan 3, 2025
22 checks passed
@Christian-B Christian-B deleted the energy_hard_reset branch January 3, 2025 11:05
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.

Energy Report vs multiple _do_run
1 participant