Skip to content

Conversation

@ocots
Copy link
Member

@ocots ocots commented Sep 26, 2025

No description provided.

@ocots ocots linked an issue Sep 26, 2025 that may be closed by this pull request
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.

🚀

@jbcaillau
Copy link
Member

@ocots i've just relaunched the demo CI 🤞🏽do you remember how long it used to take last time (= before OC.jl update)?

@ocots
Copy link
Member Author

ocots commented Sep 27, 2025

No but it seems that Goddard is very long.

@jbcaillau
Copy link
Member

WTF? if you re sure it is Goddard, let s make a separate app with it

IMG_4776

@jbcaillau
Copy link
Member

at least:

  • cost (max r(tf)) is now opposite to what it used to be after the last release (it is now positive, better, but opposite sign for H1 below)
  • ODE.jl forces some downgrades (probably because of CUDA...) on MadNLP suite; expect different results
    IMG_4778

@jbcaillau
Copy link
Member

@ocots en fait ODE.jl me brise les b....s control-toolbox/CTFlows.jl#81 (comment)

changed max r(tf) to min -r(tf) afte CTDirect update for compatibility with previous code 🤞🏽
@jbcaillau
Copy link
Member

all set:

  • max -> min -
  • grid_size 100 to 250 (maybe 150 is enough...)

merging

@jbcaillau jbcaillau merged commit c55cc97 into main Sep 27, 2025
3 checks passed
@jbcaillau jbcaillau deleted the 47-general-compat branch September 27, 2025 18:59
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.

[General] Compat

3 participants