Skip to content

Conversation

@ocots
Copy link
Member

@ocots ocots commented Feb 17, 2023

J'ai enlevé CTBase, CTDirect et CTDirectShooting pour en faire des packages.

@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #108 (3eff534) into develop (fd1d34f) will increase coverage by 20.12%.
The diff coverage is 50.00%.

❗ Current head 3eff534 differs from pull request most recent head c836f47. Consider uploading reports for the commit c836f47 to get more accurate results

@@             Coverage Diff              @@
##           develop     #108       +/-   ##
============================================
+ Coverage    59.42%   79.54%   +20.12%     
============================================
  Files           23        2       -21     
  Lines         1003       44      -959     
============================================
- Hits           596       35      -561     
+ Misses         407        9      -398     
Impacted Files Coverage Δ
src/flows.jl 77.14% <ø> (ø)
src/solve.jl 88.88% <50.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jbcaillau jbcaillau merged commit 7c88d9a into develop Feb 17, 2023
@jbcaillau jbcaillau deleted the split_in_pkgs branch February 17, 2023 15:33
"direct_shooting_CTOptimization", # unconstrained direct simple shooting
"basic",
"goddard_direct",
"goddard_indirect",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ocots normal que tu aies viré certains tests ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vu qu'il y a de nouveaux packages, les tests ont été déplacé.

constraint!(ocp, :dynamics, f)
# goddard with state constraint - maximize altitude
prob = Problem(:goddard, :state_constraint)
ocp = prob.model
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ocots vers une galerie / benchmark de problèmes ?

Copy link
Member Author

@ocots ocots Feb 17, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L'idée c'est d'avoir une librairie de problèmes pour faciliter les tests. Un problème contiendra son modèle OptimalControlModel et sa solution OptimalControlSolution. La solution peut être analytique ou calculée avec Jump, avec un de nos algos, etc. C'est plus pour du benchmark / tests unitaires que pour une galerie. Je pense que le package OptimalControl.jl aura des exemples / tutos dans sa doc et il y aura des applications plus complexes dans des repos github ce qui servira plus de galerie.

Une remarque : c'est bien d'avoir des fonctions du temps au lieu de vecteurs sur une grille de temps dans la solution d'un problème de contrôle optimal. On va harmoniser ça pour qu'à la sortie du direct ou du tir direct on ait une solution de même type qui contient des fonctions du temps pour l'état, etc.

Copy link
Member

@jbcaillau jbcaillau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ocots @PierreMartinon 👍🏽 on verra à l'usage si c'est OK de rajouter ce niveau de packages. on est d'accord que la principale motivation est de segmenter (et alléger) les tests, les builds julia...?

@ocots
Copy link
Member Author

ocots commented Feb 17, 2023

Oui ce n'est que ça a priori. Après, un développeur avancé peut éventuellement se servir d'une brique plutôt que de tout le package si nécessaire. Comme nous qui avons utiliser un petit package pour la modélisation du problème d'optimisation pour ipopt. Mais la motivation principale c'est pour alléger la compilation lors des tests. Après, à voir à l'usage. Ce sera un peu pénible tant que CTBase.jl n'est pas très stable. Il va falloir bien définir une OptimalControlSolution et un plot associé plutôt que de travailler sur les solutions de chaque méthode : directe, tir direct, indirecte...

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