View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005196 | 10000-013: Aggregates | Spec | public | 2019-10-24 18:59 | 2021-12-09 15:06 |
Reporter | Archie Miller | Assigned To | Rod Stein | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Fixed in Version | 1.05.02 RC1 | ||||
Summary | 0005196: 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 A.4.2 TimeAverage data Historian2 | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
|
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. |
|
Agreed to make this change -- needs 1.04 Errata. |
|
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. Errata changes are in 1.04.8 |
|
Agreed to 1.05 changes and 1.04.11 Errata in Virtual F2F. |
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 |