Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

[task_03] Lab3#373

Open
vimori wants to merge 48 commits into
brstu:mainfrom
vimori:task3
Open

[task_03] Lab3#373
vimori wants to merge 48 commits into
brstu:mainfrom
vimori:task3

Conversation

@vimori
Copy link
Copy Markdown
Contributor

@vimori vimori commented Dec 24, 2025

No description provided.

vimori and others added 30 commits September 26, 2025 10:37
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 24, 2025 07:49
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 24, 2025

📌 Duplicate code detection tool report

The tool analyzed your source code and found the following degree of similarity between the files:

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces Lab 3, implementing a discrete PID controller and linear model simulation system with comprehensive unit tests using Google Test.

  • Implements a discrete PID controller class with coefficients calculation and control output computation
  • Provides a linear dynamic model simulation function
  • Includes a console application for PID-based temperature control simulation

Reviewed changes

Copilot reviewed 20 out of 51 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
trunk/ii02803/task_03/src/PID.h Header file defining the discrete PID controller class with private state variables and public methods
trunk/ii02803/task_03/src/PID.cpp Implementation of PID controller with coefficient calculations and output computation
trunk/ii02803/task_03/src/lin_model.h Header file for linear dynamic model function
trunk/ii02803/task_03/src/lin_model.cpp Simple linear model implementation
trunk/ii02803/task_03/src/main.cpp Console application for PID-based control simulation with user input validation
trunk/ii02803/task_03/test/test.cpp Comprehensive unit tests covering PID controller and linear model functionality
trunk/ii02803/task_03/src/CMakeLists.txt Build configuration for main application with GoogleTest integration
trunk/ii02803/task_03/test/CMakeLists.txt Test-specific build configuration with code coverage support
trunk/ii02803/task_03/doc/report/index.md API documentation for the PID class

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread trunk/ii02803/task_03/src/PID.cpp
Comment thread trunk/ii02803/task_03/src/PID.cpp
workflow_dispatch:

permissions:
contents: read

Check notice

Code scanning / SonarCloud

Read permissions should be defined at the job level Low

Move this read permission from workflow level to job level. See more on SonarQube Cloud

permissions:
contents: read
pages: write

Check notice

Code scanning / SonarCloud

Write permissions should be defined at the job level Low

Move this write permission from workflow level to job level. See more on SonarQube Cloud
permissions:
contents: read
pages: write
id-token: write

Check notice

Code scanning / SonarCloud

Write permissions should be defined at the job level Low

Move this write permission from workflow level to job level. See more on SonarQube Cloud
Copilot AI review requested due to automatic review settings December 24, 2025 08:02
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 15 out of 42 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread trunk/ii02803/task_03/test/CMakeLists.txt
Comment thread trunk/ii02803/task_03/src/CMakeLists.txt
Comment thread trunk/ii02803/task_03/src/CMakeLists.txt
Comment thread trunk/ii02803/task_03/src/lin_model.h
Comment thread trunk/ii02803/task_03/src/PID.h
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants