nomns / nparse

Nomns' Parser for Project1999
GNU General Public License v3.0
43 stars 22 forks source link

Log event parser #48

Closed ewjax closed 1 year ago

ewjax commented 1 year ago
  1. LogEventParser functionality included
  2. DeathLoopVaccine also included
  3. The existing ParserWindow class has been broken into a Parser base class, and a ParserWindow derived class, from which the spells, maps, and discord parsers derive from as usual. Neither DeathLoopVaccine nor LogEventParser have any data to display on a window, so they both derive from Parser base class.
  4. What needs attention: storing the configuration information for LogEventParser in the config file, and reading/setting it appropriately at load time. At this point the only LogEvent data that needs to be stored is a boolean for each LogEvent class controlling whether they are included in the parsing process.
  5. A proper UI to turn all those boolean options on/off. Right now the existing code auto-magically puts a boolean yes/no toggle button on the menu for the new parsers, which is helpful for testing but probably not the desired final state.

Here is the content of the EQValet.ini file I set up, for reference:

[DeathLoopParser]
parse = True
deaths = 4
seconds = 120

[LogEventParser]
vesseldrozlin_event = True
verinatomb_event = True
dainfrostreaveriv_event = True
severilous_event = True
cazicthule_event = True
masteryael_event = True
fte_event = True
playerslain_event = True
earthquake_event = True
random_event = True
anythingbutcomms_event = False
gratss_event = True
tod_event = True
gmotd_event = True
rm-you commented 1 year ago

Can you please remake this PR except from the local repo's LogEventParser branch to the master branch? Assuming you still want to own the PR, I guess it doesn't matter but I'll let you do it. Then for future PRs, make them from ewjax/LogEventParser to nomns/LogEventParser