Skip to content

Conversation

@Zioba
Copy link
Contributor

@Zioba Zioba commented Jul 15, 2021

No description provided.

@Zioba Zioba requested review from KovalM and MikhailSadovsky July 15, 2021 15:34
const QString& contentFilename() const;
const QVariant& contentData() const;
int idtfPos() const;
QString scsId() const;
Copy link
Contributor

Choose a reason for hiding this comment

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

To discuss: why we need scs id in SCg object info

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we need to save this id, so that we can later find an element for which we will set the extended type.

see sources/plugins/scg/scs/scgobjectinforeader.cpp:99-103

static const QHash<QString, QString> exendedTypes =
{
//! not define
{ "sc_node_not_define", "node/-/-/not_define" },
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{ "sc_node_not_define", "node/-/-/not_define" },
{ SCgConsts::SC_NODE_NOT_DEFINE, "node/-/-/not_define" },

Use constants

{ "pair/var/-/temp/noorien", ">" }, //not supported
{ "pair/var/-/temp/orient", ">"}, //not supported

{ "pair/meta/-/perm/noorien", ">" }, //not supported
Copy link
Contributor

Choose a reason for hiding this comment

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

We need process types like this(without scs sign) separate, by process access arc from types tot this arc

"$$ANTLR_LIB_PATH"/runtime/src/tree/xpath/XPathTokenAnywhereElement.h \
"$$ANTLR_LIB_PATH"/runtime/src/tree/xpath/XPathTokenElement.h \
"$$ANTLR_LIB_PATH"/runtime/src/tree/xpath/XPathWildcardAnywhereElement.h \
"$$ANTLR_LIB_PATH"/runtime/src/tree/xpath/XPathWildcardElement.h \
Copy link
Contributor

Choose a reason for hiding this comment

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

KBE should depend on targets in sc-machine. And link existing (configured) targets.
Current way is very bad for support. Any changes in sc-machine will break compilation of KBE.
For that purposes we even can make scs-parser as separate target in sc-memory.

Copy link
Contributor

@deniskoronchik deniskoronchik left a comment

Choose a reason for hiding this comment

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

I still think, that:

  • efforts that already spent;
  • efforts that are still required to finish this work;
  • effort to support this in future
    are too big.

Much easier to have one simple python script that do preprocessing of knowledge base sources during knowledge base build stage. This script should do one very simple thing - convert scg to scs. There are no any problem to add call of this script into shell script that builds knowledge base.

This simple solution allows to solve this problem in some hours instead of months.

@Zioba Zioba requested a review from KovalM July 25, 2021 15:37
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.

4 participants