v3.007-dev-new (Chart refactor, OpenCL)

.clang-format

@@ -2,3 +2,4 @@
 Language: Cpp
 BasedOnStyle: Google
 ColumnLimit: 120
+IndentPPDirectives: BeforeHash

.clang-tidy

@@ -0,0 +1,108 @@
+---
+Checks: |-
+  *,
+  -altera-id-dependent-backward-branch,
+  -altera-struct-pack-align,
+  -altera-unroll-loops,
+  -bugprone-branch-clone,
+  -bugprone-copy-constructor-init,
+  -bugprone-easily-swappable-parameters,
+  -bugprone-integer-division,
+  -bugprone-macro-parentheses,
+  -bugprone-narrowing-conversions,
+  -bugprone-redundant-branch-condition,
+  -bugprone-reserved-identifier,
+  -bugprone-string-constructor,
+  -bugprone-unhandled-self-assignment,
+  -cert-dcl37-c,
+  -cert-dcl51-cpp,
+  -cert-msc30-c,
+  -cert-msc50-cpp,
+  -cert-oop54-cpp,
+  -cert-oop54-cpp,
+  -clang-diagnostic-error,
+  -concurrency-mt-unsafe,
+  -cppcoreguidelines-avoid-c-arrays,
+  -cppcoreguidelines-avoid-magic-numbers,
+  -cppcoreguidelines-avoid-non-const-global-variables,
+  -cppcoreguidelines-c-copy-assignment-signature,
+  -cppcoreguidelines-init-variables,
+  -cppcoreguidelines-macro-usage,
+  -cppcoreguidelines-narrowing-conversions,
+  -cppcoreguidelines-non-private-member-variables-in-classes,
+  -cppcoreguidelines-owning-memory,
+  -cppcoreguidelines-prefer-member-initializer,
+  -cppcoreguidelines-pro-bounds-array-to-pointer-decay,
+  -cppcoreguidelines-pro-type-member-init,
+  -cppcoreguidelines-pro-type-union-access,
+  -cppcoreguidelines-pro-type-vararg,
+  -cppcoreguidelines-special-member-functions,
+  -fuchsia-default-arguments-calls,
+  -fuchsia-default-arguments-declarations,
+  -fuchsia-overloaded-operator,
+  -fuchsia-trailing-return,
+  -google-explicit-constructor,
+  -google-global-names-in-headers,
+  -google-readability-braces-around-statements,
+  -google-readability-casting,
+  -google-runtime-int,
+  -hicpp-avoid-c-arrays,
+  -hicpp-braces-around-statements,
+  -hicpp-braces-around-statements,
+  -hicpp-exception-baseclass,
+  -hicpp-explicit-conversions,
+  -hicpp-member-init,
+  -hicpp-multiway-paths-covered,
+  -hicpp-named-parameter,
+  -hicpp-no-array-decay,
+  -hicpp-signed-bitwise,,
+  -hicpp-special-member-functions,
+  -hicpp-uppercase-literal-suffix,
+  -hicpp-use-auto,
+  -hicpp-use-equals-default,
+  -hicpp-use-nullptr,
+  -hicpp-vararg,
+  -llvm-else-after-return,
+  -llvm-header-guard,,
+  -llvmlibc-callee-namespace,
+  -llvmlibc-implementation-in-namespace,,
+  -llvmlibc-restrict-system-libc-headers,
+  -misc-definitions-in-headers,
+  -misc-no-recursion,
+  -misc-non-private-member-variables-in-classes,
+  -misc-non-private-member-variables-in-classes,
+  -misc-unconventional-assign-operator,
+  -misc-unused-parameters,,
+  -modernize-avoid-c-arrays,
+  -modernize-raw-string-literal,
+  -modernize-redundant-void-arg,
+  -modernize-use-auto,
+  -modernize-use-default-member-init,
+  -modernize-use-equals-default,
+  -modernize-use-nullptr,
+  -modernize-use-nullptr,
+  -modernize-use-trailing-return-type,
+  -modernize-use-using,
+  -performance-unnecessary-value-param,
+  -readability-avoid-const-params-in-decls,
+  -readability-braces-around-statements,
+  -readability-braces-around-statements,
+  -readability-const-return-type,
+  -readability-container-size-empty,
+  -readability-convert-member-functions-to-static,
+  -readability-delete-null-pointer,
+  -readability-else-after-return,
+  -readability-identifier-length,
+  -readability-implicit-bool-conversion,
+  -readability-isolate-declaration,
+  -readability-magic-numbers,
+  -readability-make-member-function-const,
+  -readability-named-parameter,
+  -readability-redundant-access-specifiers,
+  -readability-redundant-declaration,
+  -readability-redundant-string-init,
+  -readability-simplify-boolean-expr,
+  -readability-uppercase-literal-suffix,
+FormatStyle: file
+HeaderFilterRegex: '.*'
+WarningsAsErrors: ''

.devcontainer/Dockerfile

@@ -0,0 +1,9 @@
+# See here for image contents: https://github.com/microsoft/vscode-dev-containers/tree/v0.238.1/containers/ubuntu/.devcontainer/base.Dockerfile
+
+# [Choice] Ubuntu version (use ubuntu-22.04 or ubuntu-18.04 on local arm64/Apple Silicon): ubuntu-22.04, ubuntu-20.04, ubuntu-18.04
+ARG VARIANT="jammy"
+FROM mcr.microsoft.com/vscode/devcontainers/base:0-${VARIANT}
+
+# [Optional] Uncomment this section to install additional OS packages.
+# RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
+#     && apt-get -y install --no-install-recommends <your-package-list-here>

.devcontainer/devcontainer.json

@@ -1,14 +1,25 @@
 // For format details, see https://aka.ms/devcontainer.json. For config options, see the
 // README at: https://github.com/devcontainers/templates/tree/main/src/ubuntu
 {
+
+    // "build": {
+    	// "dockerfile": "Dockerfile",
+    	// Update 'VARIANT' to pick an Ubuntu version: jammy / ubuntu-22.04, focal / ubuntu-20.04, bionic /ubuntu-18.04
+    	// Use ubuntu-22.04 or ubuntu-18.04 on local arm64/Apple Silicon.
+    	// "args": { "VARIANT": "ubuntu-22.04" }
+    // },
+
 	// Configure tool-specific properties.
 	"customizations": {
         "vscode": {
             "extensions": [
 				"ms-vscode.cpptools",
+				"ms-vscode.cpptools-extension-pack",
+				"nicholishen.mql-over-cpp",
 				"vscodevim.vim",
 				"DavidAnson.vscode-markdownlint",
-				"EA31337.vscode-mql-tools"
+				"EA31337.vscode-mql-tools",
+				"xaver.clang-format"
 			],
         }
     },
@@ -23,7 +34,17 @@
 		// "ghcr.io/maks1ms/devcontainers-features/wine:0": {},
 		"ghcr.io/devcontainers-contrib/features/node-asdf:0": {}
 	},
+
+    // Use 'forwardPorts' to make a list of ports inside the container available locally.
+    // "forwardPorts": [],
+
+    // Use 'postCreateCommand' to run commands after the container is created.
+    // "postCreateCommand": "uname -a",
+
     // Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
     "image": "mcr.microsoft.com/devcontainers/base:jammy",
-    "name": "EA31337"
+    "name": "EA31337",
+
+   // Comment out to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
+   "remoteUser": "vscode",
 }

.github/workflows/compile-cpp.yml

@@ -13,7 +13,6 @@ on:
 jobs:
 
   FileList:
-    if: ${{ false }}  # @fixme
     outputs:
       filelist: ${{ steps.get-files.outputs.filelist }}
     runs-on: ubuntu-latest
@@ -30,7 +29,6 @@ jobs:
         run: echo ${{ steps.get-files.outputs.filelist }}
 
   Compile:
-    if: ${{ false }}  # @fixme
     runs-on: ubuntu-latest
     needs: [FileList]
     strategy:
@@ -45,9 +43,11 @@ jobs:
         with:
           compiler: gcc-latest
       - name: Compile ${{ matrix.file }} via emcc
-        run: emcc "${{ matrix.file }}"
+        if: always()
+        run: >
+          emcc -s WASM=1 -s ENVIRONMENT=node -s EXIT_RUNTIME=0 -s NO_EXIT_RUNTIME=1 -s ASSERTIONS=1 -Wall -s
+          MODULARIZE=1 -s ERROR_ON_UNDEFINED_SYMBOLS=0 --bind -s EXPORTED_FUNCTIONS="[]" -g -std=c++17
+          "${{ matrix.file }}"
       - name: Compile ${{ matrix.file }} via g++
-        run: g++ -c "${{ matrix.file }}"
-      - if: failure()
-        # Force success.
-        run: exit 0
+        if: always()
+        run: g++ -g -std=c++17 -c "${{ matrix.file }}"

.github/workflows/test-indicator.yml

@@ -35,9 +35,13 @@ jobs:
     strategy:
       matrix:
         test:
+          - Indicator.test
+          - IndicatorBase.test
           - IndicatorCandle.test
+          - IndicatorData.test
           - IndicatorTf.test
           - IndicatorTick.test
+          - IndicatorRenko.test
     steps:
       - uses: actions/download-artifact@v2
         with:

.github/workflows/test-indicators.yml

@@ -57,7 +57,6 @@ jobs:
           - Indi_ColorCandlesDaily.test
           - Indi_ColorLine.test
           - Indi_CustomMovingAverage.test
-          - Indi_DEMA.test
           - Indi_DeMarker.test
           - Indi_Demo.test
           - Indi_DetrendedPrice.test
@@ -101,6 +100,8 @@ jobs:
           - Indi_WilliamsAD.test
           - Indi_ZigZag.test
           - Indi_ZigZagColor.test
+          # Requires refactoring:
+          # - Indi_DEMA.test
     steps:
       - uses: actions/download-artifact@v2
         with:

.github/workflows/test-serializer.yml

@@ -0,0 +1,50 @@
+---
+name: Test Serializer
+
+# yamllint disable-line rule:truthy
+on:
+  pull_request:
+    paths:
+      - 'Serializer/**'
+      - '.github/workflows/test-serializer.yml'
+  push:
+    paths:
+      - 'Serializer/**'
+      - '.github/workflows/test-serializer.yml'
+
+jobs:
+
+  compile:
+    name: Compile
+    uses: ./.github/workflows/compile.yml
+    with:
+      artifact_prefix: mt
+      path: Exchange
+      skip_cleanup: true
+
+  Serializer-Tests-MQL4:
+    defaults:
+      run:
+        shell: bash
+        working-directory: serializer/tests
+    if: false
+    needs: compile
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        test:
+          - Serializer.test
+    steps:
+      - uses: actions/download-artifact@v2
+        with:
+          name: files-ex4
+      - name: Run ${{ matrix.test }}
+        uses: fx31337/mql-tester-action@master
+        with:
+          BtDays: 4-8
+          BtMonths: 1
+          BtYears: 2021
+          MtVersion: 4.0.0.1359
+          RunOnError: show_logs 200
+          TestExpert: ${{ matrix.test }}
+    timeout-minutes: 10

.github/workflows/test.yml

@@ -39,14 +39,14 @@ jobs:
     strategy:
       matrix:
         test:
+          - AccountTest
           - BufferStructTest
           - BufferTest
           - ChartTest
           - CompileIndicatorsTest
           - DatabaseTest
           # - DrawIndicatorTest
           - EATest
-          - IndicatorTest
           - IndicatorsTest
           - MailTest
           - MarketTest
@@ -60,6 +60,7 @@ jobs:
           - SummaryReportTest
           - TickerTest
           - TradeTest
+      max-parallel: 4
     steps:
       - uses: actions/download-artifact@v2
         with:
@@ -73,6 +74,7 @@ jobs:
           Version: 4
           TestExpert: ${{ matrix.test }}
           RunOnError: show_logs 200
+        if: always()
     timeout-minutes: 10
 
   Scripts-MQL4:
@@ -98,10 +100,10 @@ jobs:
           - OrderQuery
           - ProfilerTest
           - RefsTest
-          - SerializerTest
           - TerminalTest
           - TimerTest
           - ValueStorageTest
+      max-parallel: 4
     steps:
       - uses: actions/download-artifact@v2
         with:
@@ -110,6 +112,7 @@ jobs:
         uses: fx31337/mql-tester-action@master
         with:
           Script: ${{ matrix.test }}
+        if: always()
     timeout-minutes: 10
 
   Scripts-MQL4-Ignore:
@@ -136,6 +139,7 @@ jobs:
         with:
           Script: ${{ matrix.test }}
           RunOnFail: "exit 0"
+        if: always()
     timeout-minutes: 10
 
   Trade-Tests-MQL4:
@@ -158,6 +162,7 @@ jobs:
         uses: fx31337/mql-tester-action@master
         with:
           Script: ${{ matrix.test }}
+        if: always()
     timeout-minutes: 10
 
   cleanup:

.pre-commit-config.yaml

@@ -1,6 +1,12 @@
 ---
+exclude: '\.md$'
 repos:
 
+  # - repo: https://github.com/pre-commit/mirrors-clang-format
+  #   rev: v14.0.6
+  #   hooks:
+  #     - id: clang-format
+
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.0.1
     hooks:

3D/Chart3D.h

@@ -26,12 +26,12 @@
  */
 
 #include "../Bar.struct.h"
-#include "../IndicatorData.mqh"
+#include "../Indicator/IndicatorData.h"
 #include "../Indicators/Indi_MA.mqh"
 #include "../Instances.h"
 #include "../Refs.mqh"
-#include "../SerializerConverter.mqh"
-#include "../SerializerJson.mqh"
+#include "../Serializer/SerializerConverter.h"
+#include "../Serializer/SerializerJson.h"
 #include "Chart3DCandles.h"
 #include "Chart3DType.h"
 #include "Cube.h"

3D/Chart3DCandles.h

@@ -25,6 +25,7 @@
  * 3D chart candles renderer.
  */
 
+#include "../Chart.define.h"
 #include "Chart3DType.h"
 #include "Cube.h"
 #include "Device.h"