Logic-based argumentation extension of keml.analysis#7
Open
HoshankM wants to merge 16 commits intokeml-group:logic-based-argumentationfrom
Open
Logic-based argumentation extension of keml.analysis#7HoshankM wants to merge 16 commits intokeml-group:logic-based-argumentationfrom
HoshankM wants to merge 16 commits intokeml-group:logic-based-argumentationfrom
Conversation
SusieGoebel
reviewed
Mar 19, 2025
ReadMe.md
Outdated
| This project analyses KEML files statistically. For each KEML file it produces: | ||
| 1) [General Statistics](#general-statistics) | ||
| 2) [Argumentation Statistics](#argumentation-statistics) | ||
| 2) [Argumentation data](#argumentation-data) |
Member
There was a problem hiding this comment.
maybe you can find a better name? Is this about symbols?
Member
There was a problem hiding this comment.
Reading further, I think this is the whole logic part, right? you have symbols an dconstructed logical formulas. So what about logical argumentation?
| /** | ||
| * a <b>static</b> method that creates a String representation of a given {@link ArgumentTree} | ||
| * @param node {@link ArgumentTree} to be represented | ||
| * @param prefix String prefix to facilitate recursive call. Should be empty ("") during first call |
Member
There was a problem hiding this comment.
do we really need this? I think the helper is handling the prefix itself. Do you need to go back and call this method from the helper?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.