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.

Sign up for GitHub

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

Jump to bottom

make Evaluable array-only #738

Open
5 of 8 tasks
joostvanzwieten opened this issue Dec 2, 2022 · 0 comments
Open
5 of 8 tasks

make Evaluable array-only #738

joostvanzwieten opened this issue Dec 2, 2022 · 0 comments
Labels
enhancement
Milestone
v9

Comments

@joostvanzwieten
Copy link
Member

joostvanzwieten commented Dec 2, 2022 •
edited
Loading

To simplify the transition of the evaluable module to Rust we should let Evaluable be array-only.

To be removed or rewritten as an Array:

Open question:

  • How to handle functions that have multiple outputs like eig and LoopConcatenateCombined?
@joostvanzwieten joostvanzwieten added this to the v8 milestone Dec 2, 2022
@joostvanzwieten joostvanzwieten mentioned this issue Dec 2, 2022
Merged
joostvanzwieten added a commit that referenced this issue Dec 5, 2022
@joostvanzwieten
array-valued evaluable coordinate transformations (#739)
0d4cb58 
In an effort to make the `evaluable` module array-only (#738), this PR
replaces the `EvaluableTransform*` family of classes with array-valued
evaluable coordinate transformations.
@joostvanzwieten joostvanzwieten modified the milestones: v8, v9 Jul 28, 2023
@joostvanzwieten joostvanzwieten mentioned this issue Dec 20, 2023
Merged
joostvanzwieten added a commit that referenced this issue Dec 20, 2023
@joostvanzwieten
remove evaluable.Points,NPoints,Weights (#846)
9d89a1e 
In an effort to make the `evaluable` module array-only (#738), this PR replaces
and removes the `evaluable.Points`, `.NPoints` and `.Weights` classes.
@joostvanzwieten joostvanzwieten mentioned this issue Dec 20, 2023
Merged
joostvanzwieten added a commit that referenced this issue Dec 20, 2023
@joostvanzwieten
eliminate and remove EvaluableConstant (#847)
e1a61d2 
In an effort to make the `evaluable` module array-only (#738), this PR
eliminates and removes the `evaluable.EvaluableConstant` class.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
v9
Development

No branches or pull requests
1 participant
@joostvanzwieten