-
Notifications
You must be signed in to change notification settings - Fork 23
Add graph page #35
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
Add graph page #35
Conversation
I'm ok with opening one graph/grid at a time. However, do you have plans to implement the "full inspector" widget that we discussed earlier? In this case, how would we be able to dock the inspector widget in the GraphDesigner if it's not a window? 🤔 |
|
@cardinot I was planning to have a separate |
so, are you planning to open the "new graph" settings in place? (instead of a popup dialog) |
|
@cardinot I was actually planning that closing the graph would return you to the graph designer main page (the one with "New Graph" and "Open Graph") |
| * This file is part of Evoplex. | ||
| * | ||
| * Evoplex is a multi-agent system for networks. | ||
| * Copyright (C) 2018 - Marcos Cardinot <marcos@cardinot.net> |
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.
following the license template, here we write the author's name for the file; so, feel free to replace it with your name
* Add non-functional graph designer button * Add graph page * Add graph designer landing page * Remove graphpage.ui * Fix toolbar spacing * Add signals for creating graphs * Add graph designer widget * Add attribute table in Graph Designer * Remove extra attributes for now * Add cancel button * Make close button functional * 'Add button' icon and minor fixes
Simply initialize an empty page when pushing the "graph" button (uses the project button as a placeholder). This is a WIP but should be enough to give the general idea of a possible implementation.
(Solved) The commit messes up the spacing of the toolbar, but I'll fix this once I find out how Qt wspacer works exactly.
Update: 54fd499 added a simple (non yet functional) GraphDesigner which is pretty much an ExperimentDesigner with only the graph functionality present. One issue that needs to be addressed is the way attributes are handled; without an experiment we must have a way of specifying the number of attributes and perhaps other properties directly from the designer.
I opted out from making the GraphDesigner a DockWindow as I think only one graph should be able to be opened at any time and unlike the Experiment Designer, having a main page isn't necessary. But of course, I'm always open to suggestions :)