sysdiagnose icon indicating copy to clipboard operation
sysdiagnose copied to clipboard

Integrating Hackathon25 changes

Open dario-br opened this issue 8 months ago • 0 comments

Hackathon25 code changes cover the follow topics:

  1. ~~Suggested change from building tool setuptools to Hatch~~ (see pyproject.toml). The only reasoning behind was to keep dependencies only in requirements.txt file and the pyproject.toml refer to that file. However, there is another PR: #156 from @SYNchroACK to this branch that suggests to keep setuptools and simply get rid of requirements file. Potentially we can do that, it will be simpler in the end. The repo will be cleaner and we will have to update the docs.
  2. Change on Sysdiagnose class (src/sysdiagnose/__init__.py) that may break external tools that use this project as a module. Methods get_parsers and get_analysers moved to SysdiagnoseConfiguration (src/sysdiagnose/utils/base.py). This change was required by Timesketch analyser and Coverage analyser took advantage from it as well. Main class (src/main.py) was modified accordingly.
  3. Implementation of Issue #122
  4. Improvements on Issue #153
  5. New analyser: src/sysdiagnose/analysers/coverage.py that addressed #149
  6. As an example of potential diff sub-command (starts addressing Issue #155 ) there are two standalone scripts src/sysdiagnose/utils/file_stats_diff.py and src/sysdiagnose/utils/coverage_diff.py that make use of the analysers data (be aware that a decision needs to be made for coverage analyser in the output format and/or potential subcommands?) to compare the parsed data.
  7. New analyser: src/sysdiagnose/analysers/mcstateshared.py also within the topic #155 ??
  8. Implementation of Issues #87 #148 (src/sysdiagnose/__init__.py)

Pending to add PR #156 changes into this PR as well.

dario-br avatar Apr 16 '25 09:04 dario-br