Skip to content

tuto update (jb)#450

Merged
jbcaillau merged 7 commits into251-release-100from
251-release-100-jb
Apr 16, 2025
Merged

tuto update (jb)#450
jbcaillau merged 7 commits into251-release-100from
251-release-100-jb

Conversation

@jbcaillau
Copy link
Copy Markdown
Member

tutorial updates by @jbcaillau #251 (comment)

@github-actions github-actions Bot requested a review from ocots March 22, 2025 21:40
@ocots
Copy link
Copy Markdown
Member

ocots commented Apr 8, 2025

@PierreMartinon J'ai mis à jour ma branche et j'ai introduit tes modifications et celles de jb. Je fais le merge dans la branche de départ, cf #449.

Tu pourras repartir de la branche de départ. Attention, j'ai renommé et modifié le fichier anciennement oc_mam.md. Je l'ai appelé tutorial_mam.md.

Le ficher docs/src/api-ctbase/print.md doit être supprimé. Pas besoin de faire des modifs dedans.

@jbcaillau
Copy link
Copy Markdown
Member Author

@ocots @PierreMartinon salut les gars, encore bien occupé jusqu'à vendredi, dispo ensuite 🤞🏽

@PierreMartinon
Copy link
Copy Markdown
Member

@PierreMartinon J'ai mis à jour ma branche et j'ai introduit tes modifications et celles de jb. Je fais le merge dans la branche de départ, cf #449.

Tu pourras repartir de la branche de départ. Attention, j'ai renommé et modifié le fichier anciennement oc_mam.md. Je l'ai appelé tutorial_mam.md.

Le ficher docs/src/api-ctbase/print.md doit être supprimé. Pas besoin de faire des modifs dedans.

Super merci ! J'ai regle les conflits pour le futur merge, et renomme tutorial_mam en tutorial-mam pour rester coherent.
Les tests sont ok, pour la doc est-ce que tu peux me rappeler si OptimalControl doit apparaitre dans docs/Project.toml ?
J'avais cru comprendre qu'on ne le mettait pas car ca bloque un deploiement quelque part. En local, si je le dev pour l'environnement Julia principal, je peux generer la doc, et la CI Documentation semble ok egalement. On fait comme ca ?

@ocots
Copy link
Copy Markdown
Member

ocots commented Apr 9, 2025

@PierreMartinon J'ai mis à jour ma branche et j'ai introduit tes modifications et celles de jb. Je fais le merge dans la branche de départ, cf #449.
Tu pourras repartir de la branche de départ. Attention, j'ai renommé et modifié le fichier anciennement oc_mam.md. Je l'ai appelé tutorial_mam.md.
Le ficher docs/src/api-ctbase/print.md doit être supprimé. Pas besoin de faire des modifs dedans.

Super merci ! J'ai regle les conflits pour le futur merge, et renomme tutorial_mam en tutorial-mam pour rester coherent. Les tests sont ok, pour la doc est-ce que tu peux me rappeler si OptimalControl doit apparaitre dans docs/Project.toml ? J'avais cru comprendre qu'on ne le mettait pas car ca bloque un deploiement quelque part. En local, si je le dev pour l'environnement Julia principal, je peux generer la doc, et la CI Documentation semble ok egalement. On fait comme ca ?

Il ne faut pas que OptimalControl soit dans le docs/Project.toml. Pour générer la doc suivre le tuto.

@jbcaillau jbcaillau marked this pull request as ready for review April 16, 2025 16:31
@jbcaillau jbcaillau merged commit fbfdcc9 into 251-release-100 Apr 16, 2025
5 checks passed
@jbcaillau jbcaillau deleted the 251-release-100-jb branch April 16, 2025 16:32
@ocots ocots mentioned this pull request Apr 18, 2025
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