Skip to content

Merge SMOOTH_STRICT_SEC and SMOOTH_STRICT_PRIM into one analyzer #334

@hydrowoxy

Description

@hydrowoxy

Currently there are two separate analyzers for smooth strict RPM; one for PRIM, one for SEC.

The Smooth Strict RPM preset uses both of them

  {
    name: 'Smooth Strict RPM',
    description: 'Primary and Secondary RPM vs Timestamp (ms) smoothened with strict timestamp outlier removal',
    graphs: [
      {
        axes: [
          { source: 'RPM PRIM.csv', dataType: 'Timestamp (ms)' },
          { source: 'RPM PRIM.csv', dataType: 'RPM PRIM' },
        ],
        analyzer: AnalyzerType.SMOOTH_STRICT_PRIM,
        analyzerOptions: [],
        graphType: 'line',
      },
      {
        axes: [
          { source: 'RPM SEC.csv', dataType: 'Timestamp (ms)' },
          { source: 'RPM SEC.csv', dataType: 'RPM SEC' },
        ],
        analyzer: AnalyzerType.SMOOTH_STRICT_SEC,
        analyzerOptions: [],
        graphType: 'line',
      },
    ],
  },

Not sure why they are separated. I recall there was a reason but I don't remember why.

Look into a way to combine them into one SMOOTH_STRICT_RPM analyzer.

Metadata

Metadata

Assignees

Labels

backendInvolves the Java Quarkus backendcode qualityImprove the current code

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions