Merge Rabbit demo to v2 beta. - #1
Open
ChoiJangho wants to merge 57 commits into
Open
Conversation
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Rabbit designed more OOP way
2. Saved functions required to plot the rabbit under utils\plot_helper 3. Simulation Result visualization completed
Owner
Author
|
Get rid of |
Owner
Author
|
Create directory |
Owner
Author
|
Change vargin handling in |
Owner
Author
|
Refactor output handling in Change to |
2. Restructured rabbit_helper, plot, anim directory into demos/ 3. CLF-Helper rabbit version annotated more detailed.
ChoiJangho
commented
Oct 1, 2021
| % (Always preferred to refer directly to the class properties.) | ||
| params | ||
| params | ||
| config |
This function supports the dynamics that encompass reset map function.
Issue)
Should reset map, event_option be included in varargin (I think not!)
You have to define reset_map_function, event_option to rollout this function
2. run_clf_simulation_rabbit.m refined
It's pipeline got much simpler!
1) Prepare the simulation config and dynamics, event_handler
2) Initialize the state
3) Run simulation
4) Visualize the key plots
5) Run the animation
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Below updates are designed to use 'rollout_controller' for all controllers. (1) CtrlAffineSysFL/wrap_controller.m This makes controller to be compatible with rollout_controller function. Every control system that has different notion of the basic controller function should have this function to make the form compatible with the rollout_controller function (2) Suggestion We lose certain information that requires the system to evolve. ex) dVs_error, ... Those kind of information should be parsed after the evolution of the system. I suggest to use the function like 'evolve_System' instead of explicit use of ode_func By this, we can use same notion of 'wrap_evolve_system' to parse various information regardless of the control system's format This kind of update would make the code more robust as far as I am concerned. (3) version that supports both mu_ref and u_ref should be updated in future.
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
Signed-off-by: goldenhazard <octoberfor97@naver.com>
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
4 participants
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.