#5493 closed defect (fixed)
The NF does not handle array modifiers correctly
| Reported by: | Francesco Casella | Owned by: | Per Östlund |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.14.0 |
| Component: | New Instantiation | Version: | |
| Keywords: | Cc: |
Description (last modified by )
The NF doesn't yet handle array modifiers correctly. This likely the cause of a number of specific issues: #5202, #5241, #5323, #5373, #5466, #5464, #5423, #5684.
This issue is quite complex and may require some time to be resolved. When it finally is, those tickets should be checked, and then they should hopefully turn out to be resolved.
Change History (15)
comment:1 by , 5 years ago
| Summary: | The NF does not handle array modifiers correctly → The NF does not handle array modifiers involving records correctly |
|---|
comment:2 by , 5 years ago
| Description: | modified (diff) |
|---|
comment:3 by , 5 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:4 by , 5 years ago
| Milestone: | 2.0.0 → 1.16.0 |
|---|
follow-up: 6 comment:5 by , 5 years ago
@perost, what about the issues that are not been fixed? Are they due to other causes?
comment:6 by , 5 years ago
Replying to casella:
@perost, what about the issues that are not been fixed? Are they due to other causes?
Yes, #5241 and #5464 seem to be individual unrelated issues. #5323 and #5684 looks like they might be the same issue, probably record-related (Ceval errors where the argument hasn't been evaluated usually are).
I did close eight tickets though. So the number of tickets was right, if not the specific tickets.
follow-up: 8 comment:7 by , 5 years ago
Maybe also ticket #4611 falls within this ticket's scope?
In comment 7 I read:
"there is a problem with functions taking arrays of scalars as inputs and record arrays as outputs."
follow-ups: 9 11 comment:8 by , 5 years ago
Replying to ceraolo:
Maybe also ticket #4611 falls within this ticket's scope?
In comment 7 I read:
"there is a problem with functions taking arrays of scalars as inputs and record arrays as outputs."
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
follow-up: 10 comment:9 by , 5 years ago
Replying to perost:
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
The function returns a 2D array of Complex numbers, i.e., operator records.
follow-up: 13 comment:10 by , 5 years ago
Replying to casella:
Replying to perost:
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
The function returns a 2D array of Complex numbers, i.e., operator records.
I think you might've misread my comment, the last sentence was about this ticket and not #4611.
follow-up: 12 comment:11 by , 5 years ago
Replying to perost:
Replying to ceraolo:
Maybe also ticket #4611 falls within this ticket's scope?
In comment 7 I read:
"there is a problem with functions taking arrays of scalars as inputs and record arrays as outputs."
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
Ok. I'm looking forward to the moment in which developers will find time to improve the management of complex numbers by OM, so also #4611.
comment:12 by , 5 years ago
Replying to ceraolo:
Replying to perost:
Replying to ceraolo:
Maybe also ticket #4611 falls within this ticket's scope?
In comment 7 I read:
"there is a problem with functions taking arrays of scalars as inputs and record arrays as outputs."
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
Ok. I'm looking forward to the moment in which developers will find time to improve the management of complex numbers by OM, so also #4611.
#4611 is a backend issue, at least at the moment. As mentioned in the ticket there's a plan on how to handle it, and I think that's what Mahder is working on in PR #546.
comment:13 by , 5 years ago
| Summary: | The NF does not handle array modifiers involving records correctly → The NF does not handle array modifiers correctly |
|---|
Replying to perost:
Replying to casella:
Replying to perost:
No, the example model in that ticket doesn't even have any modifiers, only binding equations, so it's not related to this ticket. Actually, I'm not sure why this ticket even has "records" in the title, the issue had nothing to do with records.
The function returns a 2D array of Complex numbers, i.e., operator records.
I think you might've misread my comment, the last sentence was about this ticket and not #4611.
Of course, sorry. I'll remove the 'records' from the subject.
comment:15 by , 5 years ago
| Milestone: | 1.16.0 → 1.14.0 |
|---|
Fixed in 4cf73067.