Skip to content

Conversation

@mfogelson
Copy link
Collaborator

The P gain and the simulation time needed to be increased to match the desired performance.

NOTE: The simulations outputs NAN values for mu

storage = simulate!(mechanism, 20.0, controller!,
    record=true,
    verbose=true);
n    bvio    rvio     α       μ     |res|∞   |Δ|∞
–––––––––––––––––––––––––––––––––––––––––––––––––
1    0e+0    1e-1    1e+0    0e+0    1e-1    1e-1
2    0e+0    1e-11    1e+0    NaN     1e-11    1e-11

The P gain and the simulation time needed to be increased to match the desired performance. 

NOTE: 
```
storage = simulate!(mechanism, 20.0, controller!,
    record=true,
    verbose=true);
```
This outputs NAN values for mu
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2022

Codecov Report

Merging #59 (43da7ff) into main (2b98c76) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
- Coverage   87.67%   87.65%   -0.03%     
==========================================
  Files          94       94              
  Lines        4966     4966              
==========================================
- Hits         4354     4353       -1     
- Misses        612      613       +1     
Impacted Files Coverage Δ
src/solver/line_search.jl 95.55% <0.00%> (-1.12%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b98c76...43da7ff. Read the comment docs.

@janbruedigam janbruedigam merged commit 0fa9b88 into dojo-sim:main Nov 23, 2022
@mfogelson mfogelson deleted the mfogelson-defining-control branch June 20, 2024 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants