Skip to content

Conversation

@yuckinus
Copy link
Collaborator

@yuckinus yuckinus commented Feb 7, 2026

  • step2, single session viewer / change titlebar name from "..\continousScanning_xxxx..\session.mjs" to "..\continousScanning_xxxx\lio_result_xxx\session.mjs"
  • step1, step2 / improved console messages
  • step1 / allow folder drag'n'drop auto processing with default params (tiny spaces) (no GUI)
  • step1 / for Windows show progress in taskbar

- step2, single session viewer / change titlebar name from "..\continousScanning_xxxx\..\session.mjs" to "..\continousScanning_xxxx\lio_result_xxx\session.mjs"
- step1, step2 / improved console messages
- step1 / allow folder drag'n'drop auto processing with default params (tiny spaces) (no GUI)
- step1 / for Windows show progress in taskbar
@JanuszBedkowski
Copy link
Member

please use clang format

@yuckinus
Copy link
Collaborator Author

yuckinus commented Feb 7, 2026

sorry @JanuszBedkowski but this clang stuff is over my head. I just changed a few existing lines, added a few, can't understand what I did wrong. I'll try learning this new stuff but I don't have so much time to spare for this. just discard my changes if format is more important than content

@JanuszBedkowski
Copy link
Member

clang-format
Instalation on Linux:

sudo apt install clang-format
Installation on Windows

Go to offical llvm-project GitHub releases page
Download Windows x64 installer (for example version 21.1.8) - browser might flag exe file malware in that case mark it as "Keep it"
Run installer exe
In installation program mark "Add LLVM to the system PATH for all users" or "Add LLVM to the system PATH for all current user"
Restart any terminal or IDE that you want to use clang-format in so it reloads paths from ENV
Verification

To verify that clang-format is installed run: Verify installation by running:

clang-format --version
Formating codebase

In order to format code base run:


python3 run_clang_fromat.py

Note that PRs without formatting might be rejected from merging

If You have some issues I will help formatting it, please try and inform me if You need assist. I will help.

- bumped up version to 0.97
- switched from cout/cerr to spdlog
- improved console messages
@yuckinus
Copy link
Collaborator Author

yuckinus commented Feb 7, 2026

installed clang according to provided guideline, ran formatting commands in VS for changed files, still some files are not accepted by check_clang_format.py or Github quality check. don't understand why

@JanuszBedkowski JanuszBedkowski merged commit fb475c6 into MapsHD:main Feb 7, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants