Skip to content

Commit

Permalink
Issue #25 - Update Plot time label and Search time handling
Browse files Browse the repository at this point in the history
Update the plot x-axis label to `Time` instead of `Time (UTC)`. This
field isn't necessarily UTC since users can specify a custom field in
their telemetry that indicates time. The more generic label should help
avoid confusion here.

Update time handling in MnemonicSearch to explicitly use UTC time
instead of the default (which was changed in #3). In the future we
should attempt to update the handling of this so that the Clock mode
dictates which time format is used.
  • Loading branch information
MJJoyce committed May 2, 2018
1 parent 3887fc3 commit 9575281
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bliss/gui/static/js/bliss/gui/Plot.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ class DygraphsBackend
return {
drawPoints: true,
connectSeparatedPoints: true,
xlabel: 'Time (UTC)',
xlabel: 'Time',
labels: ['Time'],
height: 500,
width: 800,
Expand Down
2 changes: 1 addition & 1 deletion bliss/gui/static/js/bliss/gui/Search.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ const MnemonicSearch = {

let val = 'N/A'
let raw = 'N/A'
let curTime = format.datetime(new Date())
let curTime = format.datetime(new Date(), {utc: true, gps: false})
let curPacket = (bliss.packets[this._packet] ?
bliss.packets[this._packet].get(0) :
null
Expand Down

0 comments on commit 9575281

Please sign in to comment.