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

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions