Skip to content

Refactor code#48

Merged
apBUSampK merged 8 commits into
Spectator-matter-group-INR-RAS:mainfrom
ZakayZ:refactor
May 14, 2026
Merged

Refactor code#48
apBUSampK merged 8 commits into
Spectator-matter-group-INR-RAS:mainfrom
ZakayZ:refactor

Conversation

@ZakayZ
Copy link
Copy Markdown
Contributor

@ZakayZ ZakayZ commented May 14, 2026 •
edited
Loading

  • Split header (required for SWIG to work properly)
  • Loading modules as shared libs at runtime (required for build system to work with python)
  • unordered_map instead of map (we don't need order)
  • Automatic name and type deduction
  • Make function names more verbose
  • Improve performance with common constexpr functions

apBUSampK and others added 5 commits May 14, 2026 02:31
@apBUSampK @ZakayZ
Updated license
f860967 
add CMakePresets

setup clang-tidy

add lint command

update ci

fix empty flag

refactor
@ZakayZ
add module class
953cd28
@ZakayZ
add generic module
3ba57fa
@ZakayZ
adjust the style
25a1c59
@ZakayZ
add tests
cb01f9d
@ZakayZ ZakayZ force-pushed the refactor branch 2 times, most recently from 9188eb1 to e571502 Compare May 14, 2026 01:04
@alexsvetlichnyy alexsvetlichnyy self-assigned this May 14, 2026
@alexsvetlichnyy
Copy link
Copy Markdown
Member

alexsvetlichnyy commented May 14, 2026

Ok for me

@apBUSampK apBUSampK merged commit c604a0b into Spectator-matter-group-INR-RAS:main May 14, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apBUSampK apBUSampK apBUSampK approved these changes

Assignees

@alexsvetlichnyy alexsvetlichnyy

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ZakayZ @alexsvetlichnyy @apBUSampK