Skip to content

Conversation

@robinschmid
Copy link
Member

@robinschmid robinschmid commented Mar 20, 2025

Module to analyze log files - can filter out warnings and errors

identifier message severity otherValue type value
LOAD_CONFIG: io.github.mzmine.parameters.impl.SimpleParameterSet in loadValuesFromXML at 2025-03-20T13:12:23 Cannot find parameter of name AES Encryption Key in ParameterSet io.github.mzmine.gui.preferences.MZminePreferences. This might indicate changes of the parameterset and parameter types WARN    
LOAD_CONFIG: io.github.mzmine.main.impl.MZmineConfigurationImpl in getModuleParameters at 2025-03-20T13:12:24 Module class io.github.mzmine.modules.tools.batchwizard.BatchWizardModule does not provide any ParameterSet class WARN    
LOAD_CONFIG: com.sun.javafx.application.PlatformImpl in startup at 2025-03-20T13:12:24 Unsupported JavaFX configuration: classes were loaded from 'unnamed module @673fdbce' WARN    
RUN_BATCH: io.github.mzmine.util.spectraldb.parser.MonaJsonParser in parse at 2025-03-20T13:23:41 MoNA spectral library MoNA-export-LC-MS-MS_Spectra.json was imported with 1 entries failing. WARN    

logState = LogFileState.RUN_BATCH;

} else if (logState == LogFileState.RUN_BATCH && lowerMessage.startsWith(
"processing of task batch step") && lowerMessage.endsWith("finished")) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is processing of task batch step correct? i cannot find it in my logs.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2025-03-20 13:24:30 INFO io.github.mzmine.taskcontrol.impl.WrappedTask run Processing of task Batch step 24/24 done, status FINISHED

import static org.junit.jupiter.api.Assertions.assertEquals;
import org.junit.jupiter.api.Test;

class AnalyzeLogFileTaskTest {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test does not cover going out of the batch mode into manual again

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good call I will add a full log file as a resource.
I added the state tracking later

@SteffenHeu SteffenHeu merged commit d5fb022 into mzmine:master Mar 27, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants