Ticket #9227 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

AddTimeSeriesLog algorithm

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

Add an algorithm to add or extend a time series log, there will also need to be some functionality to clear (restart) a log, perhaps this is another algorithm to just delete a log.

To be tested by Russell Taylor

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 7 years ago by Martyn Gigg

  • Tester changed from Russell Talor to Russell Taylor

comment:3 Changed 7 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

comment:4 Changed 7 years ago by Martyn Gigg

Basic AddTimeSeriesLog algorithm layout + minimal tests.

Refs #9227

Changeset: 244353627adfc5114e6798fcfe09333a76bacec1

comment:5 Changed 7 years ago by Martyn Gigg

Fix small bug in DateAndTime::setFromISO8601 error reporting.

Refs #9227

Changeset: 4844da5c99436f1c1a92d9c6ba33576256b1f1ab

comment:6 Changed 7 years ago by Martyn Gigg

Create a DateTimeValidator + tests.

Refs #9227

Changeset: 9d51c79773b5e3be30f744395dd1b20545a871f2

comment:7 Changed 7 years ago by Martyn Gigg

Basic functionality works with tests.

Refs #9227

Changeset: a43f69a4fbef2d362cb38e75be7b0ba985f86c9f

comment:8 Changed 7 years ago by Martyn Gigg

Add a DeleteLog algorithm.

Refs #9227

Changeset: 9b2de3257b756b4d4c758ec31fb359e169a9745b

comment:9 Changed 7 years ago by Martyn Gigg

Fix wiki summary for AddTimeSeriesLog

Refs #9227

Changeset: c2f11f945009fc4b845bf3bc8c72081b03bcf442

comment:10 Changed 7 years ago by Martyn Gigg

Allow the type of log to be specified in AddTimeSeriesLog

Refs #9227

Changeset: f3588ba16c463732498f181558a26a9f2a362872

comment:11 Changed 7 years ago by Martyn Gigg

Add ability to wipe a whole log first before creating

Refs #9227

Changeset: d2eabc93aaa0cc481406253e98a09f866179a223

comment:12 Changed 7 years ago by Martyn Gigg

Minor refactor to improve readability.

Refs #9227

Changeset: a05712281c75c4e8bcfa1878e1ad0d41c79ef35d

comment:13 Changed 7 years ago by Martyn Gigg

Be more consistent across boost versions.

It seems older boost::date_time versions accepted just dates from a format string, whilst the newer ones do not. We'll be strict and require a time from all. Refs #9227

Changeset: 33d91840bf9b039987faeb48582666686e19695f

comment:14 Changed 7 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch: feature/9227_add_timeseries_log

Tester: There are two new algorithms: AddTimeSeriesLog and DeleteLog. The WIKI_USAGE section of AddTimeSeriesLog demonstrates a simple usage of the algorithm to add a log value in a series for the current time. This should work.

To check whether a log exists in python you can do

#! python
run = ws.run()
if "temperature" in run:
  print "Log exists"
else:
  print "Log not found"

It should also show up as a new log in the right-click "Show Sample Logs" view.

comment:15 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying

comment:16 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/9227_add_timeseries_log'

Full changeset: 15026e3ec6ab5711e20c4a9c31eb41e90e244ce5

comment:17 Changed 7 years ago by Russell Taylor

Thoroughly checked over and everything looks good.

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10070

Note: See TracTickets for help on using tickets.