OpenHeartDevelopers / CemrgApp

An Interactive Medical Imaging Platform with Image Processing and Computer Vision Toolkits for Cardiovascular Research.
http://www.cemrgapp.com
BSD 3-Clause "New" or "Revised" License
18 stars 8 forks source link

Feature/plugin/atrialfibres #68

Closed alonsoJASL closed 1 year ago

alonsoJASL commented 1 year ago

Atrial fibres plugin to create atrial models from

Allows usage of universal atrial coordinates and fibre mapping

github-actions[bot] commented 1 year ago

:zap: Code Analysis Results :zap:

:red_circle: Cppcheck found 14 issues! Click here to see details.
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Modules/CemrgAppModule/src/CemrgCommonUtils.cpp#L1679-L1684 ```diff !Line: 1679 - style: Redundant initialization for 'scalars'. The initialized value is overwritten before it is read. [redundantInitialization] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Modules/CemrgAppModule/src/CemrgCommonUtils.cpp#L1672-L1677 ```diff !Line: 1672 - note: scalars is initialized ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Modules/CemrgAppModule/src/CemrgCommonUtils.cpp#L1679-L1684 ```diff !Line: 1679 - note: scalars is overwritten ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresClipperView.h#L55-L60 ```diff !Line: 55 - style: The class 'AtrialFibresClipperView' does not have a constructor although it has private member variables. [noConstructor] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresView.h#L66-L71 ```diff !Line: 66 - style: The class 'AtrialFibresView' does not have a constructor although it has private member variables. [noConstructor] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresClipperView.cpp#L450-L455 ```diff !Line: 450 - style: Redundant initialization for 'radii'. The initialized value is overwritten before it is read. [redundantInitialization] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresClipperView.cpp#L449-L454 ```diff !Line: 449 - note: radii is initialized ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresClipperView.cpp#L450-L455 ```diff !Line: 450 - note: radii is overwritten ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresLandmarksView.h#L53-L58 ```diff !Line: 53 - style: The class 'AtrialFibresLandmarksView' does not have a constructor although it has private member variables. [noConstructor] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresLandmarksView.cpp#L392-L397 ```diff !Line: 392 - style: The scope of the variable 'distance' can be reduced. [variableScope] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresView.cpp#L1240-L1245 ```diff !Line: 1240 - error: Unmatched '('. Configuration: ''. [syntaxError] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresView.cpp#L1240-L1245 ```diff !Line: 1240 - error: Unmatched '('. Configuration: '_WIN32'. [syntaxError] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresView.cpp#L1240-L1245 ```diff !Line: 1240 - error: Unmatched '('. Configuration: '__APPLE__'. [syntaxError] ```
https://github.com/CemrgAppDevelopers/CemrgApp/blob/60117d170e946b4a1cb8ed231350623dd2f113f9/CemrgApp/Plugins/kcl.cemrgapp.atrialfibres/src/internal/AtrialFibresVisualiseView.h#L50-L55 ```diff !Line: 50 - style: The class 'AtrialFibresVisualiseView' does not have a constructor although it has private member variables. [noConstructor] ```