Skip to content
This repository has been archived by the owner on Mar 4, 2022. It is now read-only.

Longer history for graphs #64

Closed
aisapatino opened this issue Feb 21, 2017 · 1 comment
Closed

Longer history for graphs #64

aisapatino opened this issue Feb 21, 2017 · 1 comment

Comments

@aisapatino
Copy link
Member

30m for cpu/event loop/memory graphs when they're full width.
Probably don't want to store individual per-second values, so would need to add aggregation to provider.
mscottx88 pushed a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 19, 2017
feat(FormidableLabs#64) added multiple aggregate levels
f5666af
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 20, 2017
@mscottx88
fix(FormidableLabs#64) remove vscode file first, then gitignore it
22bc27a
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 20, 2017
@mscottx88
fix(FormidableLabs#64) gitignore vs code
bdd3adf
mscottx88 pushed a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 22, 2017
test(FormidableLabs#64) added unit tests for MetricsProvider
e74e992 
Also, corrected the use of sinon-chai and fixed a cross-platform issue with mock-require.
Included gulp and gulpfile for unit test debugging.
New constants.js file for configuration.
Lots of performance improvements and code cleanup in MetricsProvider aggregation logic.
Lots of documentation for MetricsProvider.
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 22, 2017
@mscottx88
test(FormidableLabs#64) removed gulp, removed duplicate setup
64e7805
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 22, 2017
@mscottx88
test(FormidableLabs#64) removed unnecessary done()
89a6fa2
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 22, 2017
@mscottx88
fix(FormidableLabs#64) test use sandbox stub
07871d1
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 22, 2017
@mscottx88
fix(FormidableLabs#64) cross-platform fix to coverage
f3b8f2c 
See gotwarlost/istanbul#90 for reasoning
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 23, 2017
@mscottx88
chore(FormidableLabs#64) simplified aggregation
6dce469
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 23, 2017
@mscottx88
chore(FormidableLabs#64) converted to inline aggregation
7b373a1
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 24, 2017
@mscottx88
test(FormidableLabs#64) 100% code coverage MetricsProvider
6dd1f3b
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 24, 2017
@mscottx88
docs(FormidableLabs#64) bumped version and added changelog entry
939ba75
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 24, 2017
@mscottx88
fix(FormidableLabs#64) corrections for code review
403853a
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 24, 2017
@mscottx88
docs(FormidableLabs#64) added notes to Changelog
b7eec6c
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 25, 2017
@mscottx88
fix(FormidableLabs#64) no more forced layouts
ae31391
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 26, 2017
@mscottx88
fix(FormidableLabs#64) improved time labels
1b61876 
Time labels now appear like digital clocks.
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 26, 2017
@mscottx88
fix(FormidableLabs#64) getTimeIndexLabel is not a member function
d749914
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 26, 2017
@mscottx88
fix(FormidableLabs#64) lodash has padStart
07ddccb
mscottx88 added a commit to mscottx88/nodejs-dashboard that referenced this issue Jul 27, 2017
@mscottx88
fix(FormidableLabs#64) memory guage view frozen
dbd091e 
The memory guage view was not getting its "metrics" events and appears frozen.
Since it is not a historical graph, the "metrics" event needed to be emitted every time.
mscottx88 pushed a commit to mscottx88/nodejs-dashboard that referenced this issue Aug 8, 2017
docs(FormidableLabs#64) separate MD for aggregation
b3f7319
mscottx88 pushed a commit to mscottx88/nodejs-dashboard that referenced this issue Aug 8, 2017
docs(FormidableLabs#64) correct PR ref
20c542f
jasonwilson added a commit that referenced this issue Aug 8, 2017
@jasonwilson
Merge pull request #66 from mscottx88/features/graph-units-of-time
4ad9351 
feat(#64) added multiple aggregate levels
@jasonwilson
Copy link
Contributor

Closed by: 4ad9351

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jasonwilson @aisapatino