View Issue Details

IDProjectCategoryView StatusLast Update
000519610000-013: AggregatesSpecpublic2021-12-09 15:06
ReporterArchie Miller Assigned ToRod Stein  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Fixed in Version1.05.02 RC1 
Summary0005196: A.1.2 Example Aggregate data – Historian 2 UseSlopedExtrapolation should be set to true
Description

The parameter UseSlopedExtrapolation should be set to true for Historian 2 queries.

The original aggregate test tool used this parameter set to do Historian 2 queries, and the spec error is leading to several miscalculations. It is also necessary to have test cases with the parameter set to true.

Additional Information

This change will result in reverting a few entries in the result tables:

A.2.2 Interpolative data Historian2
Entry 12:01:35.000 value should be reverted back to 102.5

A.4.2 TimeAverage data Historian2
Entry 12:01:30.000 value should be reverted back to 96.250
Entry 12:01:35.000 value should be reverted back to 108.750

TagsNo tags attached.
Commit Version

Relationships

related to 0005142 closedRod Stein Wrong values in Sample Historian_2 for Total 

Activities

Archie Miller

2019-10-24 19:00

administrator   ~0011160

Note from Paul Hunkar - sent April 19 2019

Not sure if you were aware of this work, but Archie is working for the certification group and generating the updated needed for the CTT to be able to calculate aggregates (and thus we would be able to implement all of the testing for aggregates). He is taking the .net code that Randy has to generate aggregates and converting it to C++ (calculation/status calc parts), since that is what the CTT needs (multi-platform and all). During his conversion he has noticed an issue – the Github project has all data for Historian2 using UseSlopedExtrapolation = true, but the specification has UseSlopedExtrapolation = false for all historians (at least that is what is in the released 1.04 spec). Is this something we changed as part of mantis issue? It would make sense to have one of the example include a sloppedExtrapolation, even though most of the aggregates would not change. If the table in the doc match what randy generate and UseSlopedExtrapolation = true I would just change the table definition to match. When I took a quick look most aggregates return Bad_NoData – so they do not use the flag. Only interpolated, TimeAverage, Total use the flag, so only the tables for these three aggregates for historian 2 for the single example that goes passed the collected time would not match what randy’s program generate. From my look at the spec it, it appears that the spec is following what is defined in it vs what the github project has.

Any thoughts would be good? We could easily change the spec since it is just an example.

Jim Luth

2019-12-12 20:29

administrator   ~0011344

Agreed to make this change -- needs 1.04 Errata.

Rod Stein

2020-09-14 01:51

developer   ~0012795

Fixed in part 13 1.05.2

Historian 2 example has been reverted to have UseSlopedExtrapolation set to true. Two locations needed to be changed.
TimeAverage 12:01:30.000 is 96.250 and Interpolative 12:01:35.000 is 102.5.

Errata changes are in 1.04.8

Jim Luth

2021-12-09 15:06

administrator   ~0015543

Agreed to 1.05 changes and 1.04.11 Errata in Virtual F2F.

Issue History

Date Modified Username Field Change
2019-10-24 18:59 Archie Miller New Issue
2019-10-24 19:00 Archie Miller Note Added: 0011160
2019-10-24 20:38 Paul Hunkar Project Specifications => 10000-013: Aggregates
2019-11-26 16:50 Rod Stein Status new => assigned
2019-11-26 16:50 Rod Stein Assigned To => Rod Stein
2019-12-12 20:29 Jim Luth Note Added: 0011344
2019-12-12 20:37 Jim Luth Relationship added related to 0005142
2020-09-14 01:51 Rod Stein Status assigned => resolved
2020-09-14 01:51 Rod Stein Resolution open => fixed
2020-09-14 01:51 Rod Stein Fixed in Version => 1.05
2020-09-14 01:51 Rod Stein Note Added: 0012795
2021-12-09 15:06 Jim Luth Status resolved => closed
2021-12-09 15:06 Jim Luth Fixed in Version => 1.05.02 RC1
2021-12-09 15:06 Jim Luth Note Added: 0015543