Skip to content
reviewdog

Implements experiments to run and time traced and untraced versions of all binaries in a project #2994

Sign in to view logs

Implements experiments to run and time traced and untraced versions of all binaries in a project

Implements experiments to run and time traced and untraced versions of all binaries in a project #2994

Triggered via pull request October 9, 2023 18:29
@niomateniomate
synchronize #588
ma-gusenburger
Status Success
Total duration 5m 2s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

reviewdog.yml

on: pull_request
reviewdog
4m 52s
reviewdog
Fit to window
Zoom out
Zoom in

Annotations

61 errors and 17 warnings
[mypy] varats/varats/plots/instrumentation_verifier_compare_experiments_overview.py#L83: varats/varats/plots/instrumentation_verifier_compare_experiments_overview.py#L83
error: invalid syntax [syntax]
[pylint] varats/varats/plots/instrumentation_verifier_compare_experiments_overview.py#L83 <1>: varats/varats/plots/instrumentation_verifier_compare_experiments_overview.py#L83
Parsing failed: 'invalid syntax (<unknown>, line 83)' (syntax-error)
[pylint] varats/varats/plots/instrumentation_verifier_overview_budget.py#L80 <1>: varats/varats/plots/instrumentation_verifier_overview_budget.py#L80
Parsing failed: 'invalid syntax (<unknown>, line 80)' (syntax-error)
[pylint] varats/varats/plots/compare_traced_budget.py#L16 <301>: varats/varats/plots/compare_traced_budget.py#L16
Line too long (101/80) (line-too-long)
[pylint] varats/varats/plots/compare_traced_budget.py#L22 <115>: varats/varats/plots/compare_traced_budget.py#L22
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/compare_traced_budget.py#L88 <115>: varats/varats/plots/compare_traced_budget.py#L88
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/compare_traced.py#L25 <115>: varats/varats/plots/compare_traced.py#L25
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/compare_traced.py#L89 <115>: varats/varats/plots/compare_traced.py#L89
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/dynamic_overhead_plot.py#L1 <114>: varats/varats/plots/dynamic_overhead_plot.py#L1
Missing module docstring (missing-module-docstring)
[pylint] varats/varats/plots/dynamic_overhead_plot.py#L17 <115>: varats/varats/plots/dynamic_overhead_plot.py#L17
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/dynamic_overhead_plot.py#L66 <115>: varats/varats/plots/dynamic_overhead_plot.py#L66
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/compare_traced_cs.py#L16 <301>: varats/varats/plots/compare_traced_cs.py#L16
Line too long (101/80) (line-too-long)
[pylint] varats/varats/plots/compare_traced_cs.py#L25 <115>: varats/varats/plots/compare_traced_cs.py#L25
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/plots/compare_traced_cs.py#L85 <115>: varats/varats/plots/compare_traced_cs.py#L85
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/projects/perf_tests/feature_perf_cs_collection.py#L92 <301>: varats/varats/projects/perf_tests/feature_perf_cs_collection.py#L92
Line too long (81/80) (line-too-long)
[pylint] varats/varats/projects/c_projects/bzip2.py#L32 <411>: varats/varats/projects/c_projects/bzip2.py#L32
third party import "from benchbuild.command import Command, SourceRoot, WorkloadSet" should be placed before "from varats.containers.containers import ImageBase, get_base_image" (wrong-import-order)
[pylint] varats/varats/projects/c_projects/bzip2.py#L32 <412>: varats/varats/projects/c_projects/bzip2.py#L32
Imports from package benchbuild are not grouped (ungrouped-imports)
[pylint] varats/varats/projects/c_projects/picosat.py#L84 <301>: varats/varats/projects/c_projects/picosat.py#L84
Line too long (93/80) (line-too-long)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L1 <114>: varats/varats/data/reports/dynamic_overhead_report.py#L1
Missing module docstring (missing-module-docstring)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L6 <115>: varats/varats/data/reports/dynamic_overhead_report.py#L6
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L10 <115>: varats/varats/data/reports/dynamic_overhead_report.py#L10
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L32 <1732>: varats/varats/data/reports/dynamic_overhead_report.py#L32
Consider using 'with' for resource-allocating operations (consider-using-with)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L2 <411>: varats/varats/data/reports/dynamic_overhead_report.py#L2
standard import "from pathlib import Path" should be placed before "from varats.report.report import BaseReport" (wrong-import-order)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L3 <411>: varats/varats/data/reports/dynamic_overhead_report.py#L3
standard import "from collections import defaultdict" should be placed before "from varats.report.report import BaseReport" (wrong-import-order)
[pylint] varats/varats/experiments/base/time_workloads.py#L83 <301>: varats/varats/experiments/base/time_workloads.py#L83
Line too long (101/80) (line-too-long)
[pylint] varats/varats/experiments/vara/instrumentation_verifier.py#L18 <301>: varats/varats/experiments/vara/instrumentation_verifier.py#L18
Line too long (92/80) (line-too-long)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L7 <301>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L7
Line too long (101/80) (line-too-long)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L1 <114>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L1
Missing module docstring (missing-module-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L16 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L16
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L29 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L29
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L43 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L43
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L57 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L57
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L71 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L71
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L84 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L84
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/dynamic_overhead_analysis.py#L97 <115>: varats/varats/experiments/vara/dynamic_overhead_analysis.py#L97
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L43 <301>: varats/varats/experiments/vara/multi_compile_experiment.py#L43
Line too long (115/80) (line-too-long)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L61 <301>: varats/varats/experiments/vara/multi_compile_experiment.py#L61
Line too long (86/80) (line-too-long)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L89 <301>: varats/varats/experiments/vara/multi_compile_experiment.py#L89
Line too long (85/80) (line-too-long)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L18 <115>: varats/varats/experiments/vara/multi_compile_experiment.py#L18
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L126 <115>: varats/varats/experiments/vara/multi_compile_experiment.py#L126
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L15 <411>: varats/varats/experiments/vara/multi_compile_experiment.py#L15
third party import "from benchbuild.experiment import Actions" should be placed before "from varats.experiment.experiment_util import Project" (wrong-import-order)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L15 <412>: varats/varats/experiments/vara/multi_compile_experiment.py#L15
Imports from package benchbuild are not grouped (ungrouped-imports)
[pylint] varats/varats/experiments/vara/feature_experiment.py#L108 <301>: varats/varats/experiments/vara/feature_experiment.py#L108
Line too long (88/80) (line-too-long)
[pylint] varats/varats/experiments/vara/feature_experiment.py#L42 <411>: varats/varats/experiments/vara/feature_experiment.py#L42
third party import "import yaml" should be placed before "from varats.experiment.experiment_util import ExperimentHandle, VersionExperiment, ZippedReportFolder, create_new_success_result_filepath, get_current_config_id, get_default_compile_error_wrapped, get_extra_config_options, WithUnlimitedStackSize" (wrong-import-order)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L18 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L18
Line too long (81/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L19 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L19
Line too long (90/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L20 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L20
Line too long (92/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L129 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L129
Line too long (82/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L138 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L138
Line too long (82/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L147 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L147
Line too long (82/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L156 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L156
Line too long (82/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L165 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L165
Line too long (84/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L219 <301>: varats/varats/experiments/vara/compare_traced_untraced.py#L219
Line too long (82/80) (line-too-long)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L34 <1101>: varats/varats/experiments/vara/compare_traced_untraced.py#L34
Instance of 'list' has no 'Step' member (no-member)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L80 <1101>: varats/varats/experiments/vara/compare_traced_untraced.py#L80
Instance of 'list' has no 'Step' member (no-member)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L129 <115>: varats/varats/experiments/vara/compare_traced_untraced.py#L129
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L138 <115>: varats/varats/experiments/vara/compare_traced_untraced.py#L138
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L147 <115>: varats/varats/experiments/vara/compare_traced_untraced.py#L147
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L156 <115>: varats/varats/experiments/vara/compare_traced_untraced.py#L156
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L165 <115>: varats/varats/experiments/vara/compare_traced_untraced.py#L165
Missing class docstring (missing-class-docstring)
[pylint] varats/varats/experiments/vara/feature_tracing_stats.py#L64 <301>: varats/varats/experiments/vara/feature_tracing_stats.py#L64
Line too long (92/80) (line-too-long)
[pylint] varats/varats/plots/compare_traced.py#L19 <511>: varats/varats/plots/compare_traced.py#L19
TODO: Is there a better way to include revisions of all workloads than to use (fixme)
[pylint] varats/varats/plots/compare_traced_cs.py#L19 <511>: varats/varats/plots/compare_traced_cs.py#L19
TODO: Is there a better way to include revisions of all workloads than to use (fixme)
[pylint] varats/varats/plots/compare_traced_cs.py#L47 <612>: varats/varats/plots/compare_traced_cs.py#L47
Unused variable 'report_file' (unused-variable)
[pylint] varats/varats/plots/compare_traced_cs.py#L10 <611>: varats/varats/plots/compare_traced_cs.py#L10
Unused get_loaded_paper_config imported from varats.paper.paper_config (unused-import)
[pylint] varats/varats/projects/c_projects/bzip2.py#L31 <404>: varats/varats/projects/c_projects/bzip2.py#L31
Reimport 'RSBinary' (imported line 14) (reimported)
[pylint] varats/varats/projects/c_projects/bzip2.py#L31 <404>: varats/varats/projects/c_projects/bzip2.py#L31
Reimport 'WorkloadCategory' (imported line 14) (reimported)
[pylint] varats/varats/projects/c_projects/bzip2.py#L32 <404>: varats/varats/projects/c_projects/bzip2.py#L32
Reimport 'Command' (imported line 6) (reimported)
[pylint] varats/varats/projects/c_projects/bzip2.py#L32 <404>: varats/varats/projects/c_projects/bzip2.py#L32
Reimport 'SourceRoot' (imported line 6) (reimported)
[pylint] varats/varats/projects/c_projects/bzip2.py#L32 <404>: varats/varats/projects/c_projects/bzip2.py#L32
Reimport 'WorkloadSet' (imported line 6) (reimported)
[pylint] varats/varats/projects/c_projects/picosat.py#L98 <511>: varats/varats/projects/c_projects/picosat.py#L98
TODO: Not sure about the categories here (fixme)
[pylint] varats/varats/data/reports/dynamic_overhead_report.py#L34 <622>: varats/varats/data/reports/dynamic_overhead_report.py#L34
Redefining built-in 'id' (redefined-builtin)
[pylint] varats/varats/experiments/vara/instrumentation_verifier.py#L70 <221>: varats/varats/experiments/vara/instrumentation_verifier.py#L70
Number of parameters was 3 in 'VaryingStartingBudgetExperiment.actions_for_project' and is now 2 in overriding 'RunInstrVerifierBudget.actions_for_project' method (arguments-differ)
[pylint] varats/varats/experiments/vara/multi_compile_experiment.py#L56 <221>: varats/varats/experiments/vara/multi_compile_experiment.py#L56
Number of parameters was 2 in 'FeatureExperiment.actions_for_project' and is now 3 in overriding 'MultiCompileExperiment.actions_for_project' method (arguments-differ)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L35 <621>: varats/varats/experiments/vara/compare_traced_untraced.py#L35
Redefining name 'actions' from outer scope (line 6) (redefined-outer-name)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L92 <621>: varats/varats/experiments/vara/compare_traced_untraced.py#L92
Redefining name 'actions' from outer scope (line 6) (redefined-outer-name)
[pylint] varats/varats/experiments/vara/compare_traced_untraced.py#L20 <611>: varats/varats/experiments/vara/compare_traced_untraced.py#L20
Unused VaryingStartingBudgetExperiment imported from varats.experiments.vara.multi_compile_experiment (unused-import)
reviewdog
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-python@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/