Skip to content

Relecture#59

Merged
ocots merged 10 commits into
mainfrom
relecture
Nov 7, 2025
Merged

Relecture#59
ocots merged 10 commits into
mainfrom
relecture

Conversation

@ocots
Copy link
Copy Markdown
Member

@ocots ocots commented Nov 2, 2025

No description provided.

@ocots ocots marked this pull request as draft November 2, 2025 13:09
@github-actions github-actions Bot requested a review from AnasXbouali November 2, 2025 13:09
@ocots
Copy link
Copy Markdown
Member Author

ocots commented Nov 2, 2025

@AnasXbouali Peux-tu relire le texte ajouté (par Claude.ai) à la documentation. Tu cliques en haut sur "Filed changed".

@ocots
Copy link
Copy Markdown
Member Author

ocots commented Nov 6, 2025

@AnasXbouali Tu vois comment rendre plus robuste le tir ?

@AnasXbouali
Copy link
Copy Markdown
Collaborator

@AnasXbouali Tu vois comment rendre plus robuste le tir ?

@ocots En modifiant la méthode directe comme suit :

sol = ( state = t -> [0.1, 0.1, 1],
               control =[-1, 0],
               variable =15)

for N in [50, 100 ,500, 1000] 
          sol = solve(ocp, :direct, :adnlp, :ipopt;
	                    disc_method = :gauss_legendre_3,
	                    grid_size=N,
	                    init=sol,
                            tol=1e-8,
	                    display=true)
end

Ça permet d’avoir des valeurs plus précises pour les temps de switch, ce qui fait converger la méthode de tir.

AnasXbouali and others added 2 commits November 7, 2025 09:35
Refactor example code for clarity and structure.
@ocots
Copy link
Copy Markdown
Member Author

ocots commented Nov 7, 2025

@AnasXbouali Je gère le souci. Il faut mettre le mot clé global dans la boucle. Ne touch à rien.

@AnasXbouali
Copy link
Copy Markdown
Collaborator

@AnasXbouali Je gère le souci. Il faut mettre le mot clé global dans la boucle. Ne touch à rien.

ok merci !

@ocots
Copy link
Copy Markdown
Member Author

ocots commented Nov 7, 2025

Je suis en train d'améliorer aussi les workflows pour ne pas toujours tout réinstaller quand on refait la documentation, etc. Car c'est très long juste pour installer les dépendances.

@ocots ocots marked this pull request as ready for review November 7, 2025 09:21
@ocots ocots merged commit 291feb3 into main Nov 7, 2025
4 checks passed
@ocots ocots deleted the relecture branch November 7, 2025 09:21
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.

2 participants