Skip to content
This repository was archived by the owner on Oct 2, 2021. It is now read-only.

Instrumentation infrastructure#303

Open
brookstalley wants to merge 11 commits intoermiyaeskandary:developfrom
brookstalley:feature/instrumentation
Open

Instrumentation infrastructure#303
brookstalley wants to merge 11 commits intoermiyaeskandary:developfrom
brookstalley:feature/instrumentation

Conversation

@brookstalley
Copy link
Copy Markdown

Description

  • Adds logUtil, and easy way to profile performance and get statistics including mean, stdev, min, and max
  • Adds stdev stat to bot.scores display
  • Adds indication of last X (default 200) frames performance from computeFoodGoal

TESTING STAGE

Should not change actual bot performance.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read CONTRIBUTING.md
  • I fully understand the Github Flow and I'm merging into the develop branch , which is the default branch, and not the master branch
  • My code adheres to the code style of this project but most importantly of all to the things mentioned, which I have read, in CONTRIBUTING.md
  • If my change requires a change to the documentation, I have updated the documentation in /docs accordingly.
  • I have included the results of the test required.

oContent.push('a: ' + Math.round(
bot.scores.reduce(function(a, b) { return a + b; }) / (bot.scores.length)) +
' m: ' + median);
oContent.push('avg: ' + avg + ' stdev: ' + stdev + '<br/>' +
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'avg' is not defined.
'stdev' is not defined.

@K00sKlust
Copy link
Copy Markdown
Collaborator

K00sKlust commented Jun 12, 2016

👍

Approved with PullApprove

bot.sectorBoxSide, bot.sectorBoxSide);
// if (window.visualDebugging) canvasUtil.drawRect(bot.sectorBox, '#c0c0c0', true, 0.1);
// if (window.visualDebugging)
// canvasUtil.drawRect(bot.sectorBox, '#c0c0c0', true, 0.1);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better to delete this if it's unused.

@ChadSki
Copy link
Copy Markdown
Collaborator

ChadSki commented Jun 17, 2016

I like it! Need to fix some minor issues and then I will approve.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants