Skip to content

Gui/fix logger #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
chayaRoten wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Gui/fix logger #61

chayaRoten wants to merge 33 commits into from

Conversation

@chayaRoten
Copy link
Collaborator

@chayaRoten chayaRoten commented Sep 18, 2024

Overview
This pull request introduces significant changes to the logging system in the project, focusing on code conventions and the organization of log files.

Changes Made

1. Code Conventions:

  • Updated the logger files to adhere to established coding standards, ensuring consistency and readability across the codebase.
  • Applied formatting and style guidelines to improve code quality and maintainability.

2. Log File Organization:

  • Refactored the implementation so that log files are now created within a dedicated and structured directory.
  • This change enhances the management of log files, making it easier to locate and handle them.

Benefits

  • Improved Readability: Following coding conventions enhances the clarity of the code, making it easier for current and future developers to understand.
  • Better Organization: Storing log files in a specific directory prevents clutter in the main project structure and allows for more efficient log management.

ChedvaErblich and others added 30 commits September 14, 2024 21:07
@ChedvaErblich
GUI: user interface
9c11401
@RivkiYagelnik @ChedvaErblich
GUI: Update .gitignore and remove ignored files from tracking
1501851
@chayaRoten @ChedvaErblich
GUI: Modified to ensure processes terminate and allowed the user to s…
480f528 
…elect QEMU from predefined options
@Tamar-Leibovitz @ChedvaErblich
GUI: Make all proccesses run together and fix the end button
be598ae
@ChedvaErblich
GUI: Background completion on the entire window
ef7feee
@DvoraNovogrotzki @ChedvaErblich
GUI: Connect the Binary Json to the GUI
86515f3
@RivkiYagelnik @ChedvaErblich
GUI: Log handler with update about this in mainWindow
644faf2
@RivkiYagelnik @ChedvaErblich
GUI: Add copy ctor's dragbleSquer and Process. Add members to log_han…
4d44f07 
…dler
@RivkiYagelnik @ChedvaErblich
GUI: Update the process map according to information from the JSON file
23b2c9a
@RivkiYagelnik @ChedvaErblich
GUI: Completing the log handler and creating frames
4e63353
@chayaRoten @ChedvaErblich
GUI: Refactored MainWindow: Add deletion functionality
816e251
@chayaRoten @ChedvaErblich
GUI: Add editing functionality
5ea4a58
@chayaRoten @ChedvaErblich
GUI: Enable running script files
33a4199
@DvoraNovogrotzki @ChedvaErblich
GUI: Dynamic frames with information from log file
ca2e514
@Tamar-Leibovitz @ChedvaErblich
GUI: Finish with all test functions
23b0698
@s7932 @ChedvaErblich
GUI: Modify code to create project configuration file with project ID
0039898
@ChedvaErblich
GUI: Move all source files to gui directory and add include directory
451284d
@RivkiYagelnik @ChedvaErblich
GUI: Omitting an unnecessary function named findProcessCoordinatesById
072b579
@RivkiYagelnik @ChedvaErblich
GUI: Registering for the logger and using it in MainWindow
62fdb49
@RivkiYagelnik @ChedvaErblich
GUI: Insertion of error debug sentences and information throughout th…
9c781fc 
…e project

Additionally, organized the code according to the agreed conventions.
@ChedvaErblich
GUI: Separate compile and run processes
5a6f5b6
@s7932 @ChedvaErblich
GUI: Allow stopping a single process while running
181f49a
@RivkiYagelnik @ChedvaErblich
GUI: Change styleSheet to squers
f5abe19
@RivkiYagelnik @ChedvaErblich
GUI: Add test to log_handler
b894ccd
@ChedvaErblich
GUI: Fix test functions and update CMakeLists.txt file
847d14c
@DvoraNovogrotzki @ChedvaErblich
GUI: Add state save and load functionality
91cbfba 
- Implemented the 'saveSimulationState' function to save simulation state to a BSON file.
- Added 'updateStateFromJson' to load simulation state from JSON data.
- Fixed issues with process handling by using Process pointers correctly.
- Updated 'DraggableSquare' integration to support new state management logic.
@DvoraNovogrotzki @ChedvaErblich
GUI: Update Square Frames visualisation
3d31e12
@DvoraNovogrotzki @ChedvaErblich
GUI: Add load and show simulation buttons
b40dcdc
@ChedvaErblich
GUI: Support for CTest and individual test execution
29c6fa3
@chayaRoten @ChedvaErblich
GUI:Update executable path option and add file selection UI
cdb6e59
@chayaRoten chayaRoten changed the base branch from main to GUI_main September 18, 2024 11:51
@ChedvaErblich ChedvaErblich force-pushed the GUI_main branch 2 times, most recently from e999385 to ca5bb0a Compare September 23, 2024 13:57
Base automatically changed from GUI_main to main September 24, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@chayaRoten @ChedvaErblich @RivkiYagelnik @Tamar-Leibovitz @DvoraNovogrotzki @s7932