sebse/factorio-signal-exporter
1
0
Fork 0
Code Issues 25 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix: pass config.series_limit to buildSeriesData instead of hardcoded 80

Browse Source
This commit is contained in:
Sebastian Seedorf
2026-06-05 12:48:47 +02:00
parent dc5ded77d5
commit 8b5d4ee6a5
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
web/components/ChartCard/SignalsChart.tsx
View File

@@ -50,7 +50,7 @@ export default function SignalsChart({
const { containerRef, legendRef } = usePlot( const { containerRef, legendRef } = usePlot(
(el, w, h, lRef) => { (el, w, h, lRef) => {
const data = buildSeriesData(rows, config.signal_type, timeMode); const data = buildSeriesData(rows, config.signal_type, timeMode, config.series_limit);
if (!data) return null; if (!data) return null;
const { keys, allXs, data: seriesData } = data; const { keys, allXs, data: seriesData } = data;

5
web/components/ChartCard/seriesData.ts
View File

@@ -1,7 +1,5 @@
import type { SignalRow, ChartConfig, TimeMode } from '@/lib/types'; import type { SignalRow, ChartConfig, TimeMode } from '@/lib/types';
const MAX_SERIES = 80;
export interface SeriesData { export interface SeriesData {
keys: string[]; keys: string[];
allXs: number[]; allXs: number[];
@@ -12,6 +10,7 @@ export function buildSeriesData(
rows: SignalRow[], rows: SignalRow[],
signalType: ChartConfig['signal_type'], signalType: ChartConfig['signal_type'],
timeMode: TimeMode, timeMode: TimeMode,
seriesLimit: number = 80,
): SeriesData | null { ): SeriesData | null {
const seriesMap = new Map<string, Map<number, number>>(); const seriesMap = new Map<string, Map<number, number>>();
@@ -34,7 +33,7 @@ export function buildSeriesData(
if (seriesMap.size === 0) return null; if (seriesMap.size === 0) return null;
const keys = [...seriesMap.keys()].slice(0, MAX_SERIES); const keys = [...seriesMap.keys()].slice(0, seriesLimit);
const allXs = [ const allXs = [
...new Set( ...new Set(
keys.flatMap((k) => { keys.flatMap((k) => {