-
Notifications
You must be signed in to change notification settings - Fork 30
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
Issues 75 and 94 #113
Open
morosz
wants to merge
61
commits into
codemetropolis:develop
Choose a base branch
from
zakorgy:M2
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Issues 75 and 94 #113
Conversation
This file contains 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
…of the source code elements and the attributes of the buildable types.
…ile editor. There is a known issue! See below for details. I have also created the class for the editor dialog and made some minor changes on the BuildableSettings and PropertyCollector classes. Known issue: in CodeMetropolisGUI.java you have to change the mappingEditorCdfPath.getText() in the line 215, as written in the comment. If you don't do that you will get the error message that the file you have just chosen does not exist. If you replace the parameter, it will work just fine. (An empty dialog should pop up without occuring any kind of exception.)
Added resource creating options. Added saving options. Added tabbed pane of the buildable elements. Other minor changes.
…Added file chooser to specify the mapping file saving path.
…me. Added classes which reprezent the various types of conversions.
Add drag and drop feature to mapping file editor.
Fix invalid column dropping and invalid builtin attribute and metric assign. Add quantization dialog class and add quantization class. Extend mapping editor dialog: - add conversion data members - add tabels getters - add action listener to conversion button, but there is not effect. Fix typo NormalizeConversion class name.
Add unit test to typeChecker method in TransferHelper class. Add unit test to validateProperties method in BuildableSettings class.
…rsion, some refactoring.
Fix nullPointExeption error in typeChecker meethod. Add new test cases to Transferhelper class typeChecker method. Add new test case BuidableSettings class validateProperties method. Add new test case QuantizationInformation class.
…onverterToMapping.xml.
…converter's method.
…verter Feature/morosz/browsing history converter
- Now resources can be assigned to buildable properties. - In case of bad formatted or missing coniguration file the editor will display the default list of buildable properties.
Feature/acsokosok/guiextension
- Fixing the error of the drag 'n drop feature. - Using TableModelListener for setting/updating conversions. - Creating a custom table class, which can store the conversions related to it (CMTable.java). - New way to store conversions instead of the silly method of class hierarchy. Conversions are stored as Objects from now. - Removed unnecessary conversion classes. The class QuantizationConversion has got a setter method for its list attribute. - Rewriting from scratch the whole dialog class which is responsible for setting the parameters (levels) of a quantization conversion. - Removing unnecessary and/or compex code snippets. - Added new translations for the gui.
- Fix typo in javadoc comment in CodeMetropolisGUI.java - Listener actions of the mapping fie editor dialog moved to seperate listener classes. - Removed unnecessary GUI components from the mapping file editor dialog. - Extend CMTable class with 'source' and 'target' attributes. - Extend BrowseListener.java with new constructor. - New GUI translations, removed unnecessary ones. - Added listener class for creating the output mapping file. - ISSUE: you cannot specify the path for the output trough the GUI. You have to specify it in the code!
Feature/acsokosok/guiext/plus
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.
We have been added two converters: Browsing History and Git-inspector converter and we improved the GUI with a mapping file editor window. The converters work similar as the currently supported converters. Browsing History converter allows to visualize a user's browsing history, based on an exported xml file. The other converter: Git-inspector converter, also uses xml files, to generate the CDF tree. This converter visualize Git metrics. Both converter uses unique mapping files.
This pull request is submitted as a solution to these issues: #75, #94