Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

287 dev avoid generic types in struct #294

Merged
merged 6 commits into from
Sep 14, 2024

Conversation

ocots
Copy link
Member

@ocots ocots commented Sep 10, 2024

No description provided.

@ocots ocots linked an issue Sep 10, 2024 that may be closed by this pull request
Copy link

Package name latest stable
CTDirect.jl compat: v0.13.1 compat: v0.13.1
CTFlows.jl compat: v0.13.1 compat: v0.13.1
OptimalControl.jl compat: v0.13.1 compat: v0.13.1

@ocots
Copy link
Member Author

ocots commented Sep 10, 2024

@jbcaillau can you have a look?

@ocots ocots marked this pull request as draft September 11, 2024 15:52
@ocots ocots marked this pull request as ready for review September 11, 2024 19:07
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 what does v2 to v4 change do?

@ocots
Copy link
Member Author

ocots commented Sep 14, 2024

@ocots what does v2 to v4 change do?

v2 is deprecated and not accepted anymore. No run with v2. But with v4, we have a bug :-(

@ocots
Copy link
Member Author

ocots commented Sep 14, 2024

@jbcaillau I have:

  • made parametric types for callable types;
  • reviewed the constructors with DataTypes.

I think we can merge and we will see later to fix the bug for breakage.

@ocots ocots merged commit bbec4f5 into main Sep 14, 2024
4 of 9 checks passed
@ocots ocots deleted the 287-dev-avoid-generic-types-in-struct branch September 14, 2024 20:20
@jbcaillau
Copy link
Member

jbcaillau commented Sep 17, 2024

@jbcaillau can you have a look?

@ocots busy these days, sorry I could not have a finer look at this. regarding what we discussed, if adding the {<:Function, ...} type parameter does not complicate too much calling the construction (we want to avoid sth like Cons{typeof(f), ...}(f)!), good 🤞🏽

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.

[Dev] Avoid generic types in struct
2 participants