Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor/remove codegen #65

Merged
merged 26 commits into from
Dec 11, 2024
Merged

Refactor/remove codegen #65

merged 26 commits into from
Dec 11, 2024

Conversation

MakarenkoAI
Copy link
Collaborator

No description provided.

@MakarenkoAI MakarenkoAI changed the title Fix/new api Refactor/remove codegen Sep 17, 2024
@MakarenkoAI MakarenkoAI self-assigned this Sep 17, 2024
kilativ-dotcom
kilativ-dotcom requested changes Sep 18, 2024
View reviewed changes
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
...rm-dependent-components/problem-solver/cxx/sections-module/agent/get_decomposition_agent.cpp Outdated Show resolved Hide resolved
...rm-dependent-components/problem-solver/cxx/sections-module/agent/get_decomposition_agent.cpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/agent/remove_section_agent.cpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/agent/remove_section_agent.cpp Outdated Show resolved Hide resolved
...ependent-components/problem-solver/cxx/sections-module/test/units/add_section_agent_test.cpp Outdated Show resolved Hide resolved
kilativ-dotcom
kilativ-dotcom approved these changes Sep 19, 2024
View reviewed changes
Copy link
Member

@kilativ-dotcom kilativ-dotcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix comments and it'll look ok 👍

...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/agent/add_section_agent.cpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.cpp Outdated Show resolved Hide resolved
Malina09
Malina09 reviewed Nov 4, 2024
View reviewed changes
Copy link
Collaborator

@Malina09 Malina09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had the system not build because of this mismatched naming, I think that's why ci doesn't pass.

...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.hpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.hpp Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.hpp Outdated Show resolved Hide resolved
kilativ-dotcom
kilativ-dotcom reviewed Nov 11, 2024
View reviewed changes
scripts/build_kb.sh Outdated Show resolved Hide resolved
NikitaZotov
NikitaZotov requested changes Nov 21, 2024
View reviewed changes
CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
...em-solver/cxx/identifiers-module/agent/translate_main_system_idtfs_from_sc_to_file_agent.hpp Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/identifiers-module/identifiers_module.cpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/identifiers-module/identifiers_module.hpp Outdated Show resolved Hide resolved
...ture_with_context/ui_menu_file_for_counting_structures_in_structure_with_context_content.scs Outdated Show resolved Hide resolved
scripts/run_py_ps.sh Outdated Show resolved Hide resolved
scripts/set_vars.sh Outdated Show resolved Hide resolved
scripts/set_vars.sh Outdated Show resolved Hide resolved
scripts/run_sc_web.sh Outdated Show resolved Hide resolved
NikitaZotov
NikitaZotov requested changes Nov 26, 2024
View reviewed changes
...ndent-components/problem-solver/cxx/identifiers-module/test/test-structures/correct_test.scs Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/agent/add_section_agent.cpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/agent/remove_section_agent.cpp Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/keynodes/sections_keynodes.hpp Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/test/tests.cmake Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/utils/sections_utils.hpp Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/sections-module/utils/set_utils.cpp Outdated Show resolved Hide resolved
scripts/run_sc_web.sh Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/identifiers-module/CMakeLists.txt Outdated Show resolved Hide resolved
platform-dependent-components/problem-solver/cxx/identifiers-module/test/tests.cmake Show resolved Hide resolved
NikitaZotov
NikitaZotov approved these changes Dec 3, 2024
View reviewed changes
Copy link
Member

@NikitaZotov NikitaZotov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge for now, we'll fix the build on macOS when we move project to the new build system

MakarenkoAI and others added 19 commits December 8, 2024 21:05
@MakarenkoAI
refactor: update metasystem, fix warnings
80ece5b
@MakarenkoAI
refactor: change ScTypes on the new format
a910497
@Malina09 @MakarenkoAI
fix: scweb repo path
f69443b
@MakarenkoAI
refactor: update scs types
b98404a
@MakarenkoAI
fix(script): set_vars
6f1b28a
@MakarenkoAI
fix(scripts): set_vars
636dd94
@MakarenkoAI
fix(kb): update scs types
af2f35f
@MakarenkoAI
fix(scripts): use sc-machine with scripts
d24c5d0
@MakarenkoAI
fix(config): update ini-file
101f53e
@MakarenkoAI
fix(config): activate env for sc-web and py-server
00ace2f
@Malina09 @MakarenkoAI
fix(kb): fix lang_en in user interface
ee88e61
@MakarenkoAI
fix(kb): remove action from ui menu
484fc8b
@MakarenkoAI
fix(kb): remove action from tests in scs files
19c7c66
@MakarenkoAI
fix(scripts): update sc-machine and sc-web commits
c97a1e5
@MakarenkoAI
fix: add sc-core dependency in tests.cmake
cf8f625
@MakarenkoAI
fix(cmake): add cmake-flags
15713d4
@NikitaZotov @MakarenkoAI
refactor: sort includes
d2dab86
@MakarenkoAI
feat(docs): update changelog
d8dd2f1
@MakarenkoAI
refactor(ps): use clang-formatter
367c730
Malina09
Malina09 approved these changes Dec 9, 2024
View reviewed changes
Copy link
Collaborator

@Malina09 Malina09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks well!

@MakarenkoAI MakarenkoAI merged commit 46a7ce9 into main Dec 11, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kilativ-dotcom kilativ-dotcom kilativ-dotcom approved these changes

@Malina09 Malina09 Malina09 approved these changes

@NikitaZotov NikitaZotov NikitaZotov approved these changes

Assignees

@MakarenkoAI MakarenkoAI

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MakarenkoAI @NikitaZotov @kilativ-dotcom @Malina09