Skip to content

Refactor: Restructure codebase - move C++ to src/cpp, rename bindings to python - #360

Merged
antond-weta merged 4 commits into
AcademySoftwareFoundation:mainfrom
soswow:structure-refactor
Oct 18, 2026
Merged

antond-weta merged 4 commits into
AcademySoftwareFoundation:mainfrom
soswow:structure-refactor

Conversation

@soswow

@soswow soswow commented Oct 17, 2026

Copy link
Copy Markdown
Contributor
  • Move all C++ code from /src to /src/cpp (preserving subfolder structure)
  • Move C++ public headers from /include/rawtoaces/ to /src/cpp/include/ (flat structure, no rawtoaces/ subdirectory)
  • Rename /src/bindings to /src/python
  • Update all #include statements to remove rawtoaces/ prefix
  • Update CMakeLists.txt files to reference new include paths
  • Update AGENTS.md, CONTRIBUTING.md, generate_python_docs.py to reference python instead of bindings
  • Update tests to use new paths

Assisted-by: OpenCode / Qwen3-Coder-Next

… to python

- Move all C++ code from /src to /src/cpp (preserving subfolder structure)
- Move C++ public headers from /include/rawtoaces/ to /src/cpp/include/ (flat structure, no rawtoaces/ subdirectory)
- Rename /src/bindings to /src/python
- Update all #include statements to remove rawtoaces/ prefix
- Update CMakeLists.txt files to reference new include paths
- Update AGENTS.md, CONTRIBUTING.md, generate_python_docs.py to reference python instead of bindings
- Update tests to use new paths

Assisted-by: OpenCode / Qwen3-Coder-Next
Signed-off-by: Aleksandr Motsjonov <soswow@gmail.com>
Co-authored-by: OpenCode <opencode@opencode.ai>
@codecov-commenter

codecov-commenter commented Oct 17, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.79%. Comparing base (360707c) to head (23b7380).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #360      +/-   ##
==========================================
+ Coverage   91.76%   91.79%   +0.02%     
==========================================
  Files          17       17              
  Lines        3876     3876              
  Branches      582      582              
==========================================
+ Hits         3557     3558       +1     
+ Misses        319      318       -1     
Files with missing lines Coverage Δ
src/cpp/include/rawtoaces/image_converter.h 100.00% <ø> (ø)
src/cpp/include/rawtoaces/rawtoaces_core.h 100.00% <ø> (ø)
src/cpp/include/rawtoaces/spectral_data.h 100.00% <ø> (ø)
src/cpp/include/rawtoaces/usage_timer.h 100.00% <ø> (ø)
src/cpp/rawtoaces/main.cpp 100.00% <ø> (ø)
src/cpp/rawtoaces_core/core_math.cpp 99.80% <ø> (ø)
src/cpp/rawtoaces_core/define.h 100.00% <ø> (ø)
src/cpp/rawtoaces_core/rawtoaces_core.cpp 98.50% <ø> (ø)
src/cpp/rawtoaces_core/spectral_data.cpp 100.00% <ø> (ø)
src/cpp/rawtoaces_util/cache_base.h 100.00% <ø> (ø)
... and 7 more

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 360707c...23b7380. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

soswow and others added 2 commits October 18, 2026 19:07
… to src/cpp/bindings.

Signed-off-by: Aleksandr Motsjonov <soswow@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@soswow
soswow force-pushed the structure-refactor branch from 3981057 to 190a370 Compare October 18, 2026 08:08
@antond-weta
antond-weta merged commit 5d07dda into AcademySoftwareFoundation:main Oct 18, 2026
29 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.

3 participants