Skip to content

Report generators#65

Merged
alex98247 merged 12 commits intomainfrom
feature/report-generators
Jul 28, 2025
Merged

Report generators#65
alex98247 merged 12 commits intomainfrom
feature/report-generators

Conversation

@quepum
Copy link
Contributor

@quepum quepum commented Jul 25, 2025

Summary

Implement a report generation system for experiments using HTML templates, add report builders supporting tables and plots.

Solve the issue: #64

Quick changelog

  • Implemented report builders CriticalValueReportBuilder, TimeComplexityReportBuilder, PowerReportBuilder.
  • Created HTML templates for each report type.
  • Added CLI option to choose report type (with/without charts).
  • Added unit tests for all report builders.

What's new?

This PR introduces a flexible and scalable system for creating reports from the results of experiments.

@quepum quepum added the python Pull requests that update python code label Jul 25, 2025
Copy link
Contributor

@f1i3g3 f1i3g3 left a comment

Choose a reason for hiding this comment

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

Надо подправить под CI

@quepum quepum requested a review from f1i3g3 July 25, 2025 11:58
Copy link
Contributor

@f1i3g3 f1i3g3 left a comment

Choose a reason for hiding this comment

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

Все круто!
Только я с телефона посмотреть нормально не могу, но верю, что тесты нормальные и проходятся.
Прошу остальных тоже глянуть.
Отдельно - наверное, примеры готовых html лучше скинуть в чат/залить в репу (или я просто не вижу с телефона).

Copy link
Contributor

Choose a reason for hiding this comment

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

Возможно, в будущем надо будет расширять - но пока что ок

self.criteria_config = criteria_config
self.significance_levels = significance_levels
self.sizes = sample_sizes
self.sizes = sorted(sample_sizes)
Copy link
Contributor

Choose a reason for hiding this comment

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

А зачем тут сортировка?

experiment_steps.execution_step = execution_step

report_building_step = self._create_report_building_step(result_storage)
report_building_step = self._create_report_building_step(result_storage, experiment_storage)
Copy link
Contributor

Choose a reason for hiding this comment

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

Так и должно быть?

@alex98247 alex98247 merged commit 1df2d79 into main Jul 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants