-
Notifications
You must be signed in to change notification settings - Fork 468
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
Improve the running speed of the test and application modules #120
Open
fantasystarlight
wants to merge
18
commits into
BIMK:master
Choose a base branch
from
fantasystarlight:master
base: master
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.
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
set the initialized property of PROBLEM to 1 after the algorithm runs one time
Add an initialized property for PROBLEM, it will be set to 1 after the algorithm runs one time Add the argument PROBLEM for the DRAW function If the PROBLEM.initialized==1, the true PF does not need to be draw again
Add the PROBLEM as a new argument for Draw() If the PROBLEM.initialized == 0, clear the plot If more than one children in ax, clear the plot of the solutions and keep PF, otherwise clear the plot If the PROBLEM.initialized == 0, initialize the settings of the plot, otherwise skip the settings
Add the PROBLEM argument for Draw()
Add the PROBLEM argument for Draw()
Add the PROBLEM argument for Draw()
Thank you for your advice, we'll check it as soon as possible. |
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.
I think the settings of the plot and the PF do not need to update at each iteration, these operations are time-consuming.
So, I add two properties for PROBLEM, fastdraw and display, where fastdraw is 0 or 1, and display is a string of the current display item, and add the PROBLEM as a new argument for Draw()
PROBLEM.fastdraw=0 when the PROBLEM objective is created. The plot will be updated by the usual method if PROBLEM.fastdraw=0.
When the PROBLEM.fastdraw=1, the Draw() will only update the plot of solutions, the settings of the plots and the PF will not be updated.
The purpose of PROBLEM.display is to detect the change in the display item, it will be updated after the plot has been drawn at each iteration. At each iteration before the Draw() running, If the current display item is not the same as PROBLEM.display, that means that the display item has been changed, then the PROBLEM.fastdraw will be set to 0. After the plot has been drawn, the PROBLEM.fastdraw will be set to 1. In the following iteration, if the display item is not changed, only the plot of solutions will be updated.
I have debugged these codes, and it's significantly faster than the original. If these codes have any bugs, please let me know, thank you.