-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Remove deprecated types #74
base: main
Are you sure you want to change the base?
Remove deprecated types #74
Conversation
@@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
### Breaking changes | |||
- Direct inference agent's subscription element is changed to `action_initiated` | |||
|
|||
### Changed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about changed types?
GenerationUtils::generateRelationBetween(ms_context, pair, replacement, ScKeynodes::rrel_1); | ||
GenerationUtils::generateRelationBetween(ms_context, pair, variable, ScKeynodes::rrel_2); | ||
ms_context->GenerateConnector(ScType::EdgeAccessConstPosTemp, variable, replacement); | ||
ms_context->GenerateConnector(ScType::ConstTempPosArc, variable, replacement); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can specify actuality here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resee everywhere please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so
for one formula this substitution is actual, but there is a lot of solution trees in the kb and variables may have different substitutions at every formula usage
@@ -28,7 +28,7 @@ class DeleteSolutionManager | |||
|
|||
ScAddrList getListFromSet(ScAddr const & set) const; | |||
|
|||
void deleteEdges(ScAddr const & source, ScType const & edge, ScAddr const & target) const; | |||
void deleteConnectors(ScAddr const & source, ScType const & connectorType, ScAddr const & target) const; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
void deleteConnectors(ScAddr const & source, ScType const & connectorType, ScAddr const & target) const; | |
void eraseConnectors(ScAddr const & source, ScType const & connectorType, ScAddr const & target) const; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check everywhere such prefixes as create, generate, erase, delete, remove, get, search, find
No description provided.