-
Notifications
You must be signed in to change notification settings - Fork 19
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
Create Super-Module for LibSvm/Liblinear #434
Comments
LibX :)
Tobias Horsmann <[email protected]> schrieb am Fr. 2. Feb. 2018 um
21:36:
… Both modules use the same data format and could use a shared super-module.
This would reduce code redundancy considerably. Except for the actual API
calls to the backend-SVM, the TC Code is identical (in behavior not in
actual line-of-codes).
@zesch <https://github.com/zesch> any idea for a new module name for this
middle-hierarchy module ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#434>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACkQ4NgfE3z0Ckhf1P4CoMxIQA6kbC5sks5tQ3HqgaJpZM4R3vsm>
.
|
Isn't a shared module that is used by two other modules lower in the hierarchy? |
Hm, depends how you look at it. If you think of the TC project as a (computer science) tree then the leafs are the lowest level, i.e. most specific, with this LibX project being an intermediate branch between the root and the leaf for the LibSvm and Liblinear project? It might be possible to let SvmHmm use the same top-level project and #428 might benefit from it, too. Some name that express |
how about |
I suppose in DKPro Core, it would be called |
Both modules use the same data format and could use a shared super-module.
This would reduce code redundancy considerably. Except for the actual API calls to the backend-SVM, the TC Code is identical (in behavior not in actual line-of-codes).
@zesch any idea for a new module name for this middle-hierarchy module ?
The text was updated successfully, but these errors were encountered: