Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

Curve points are limited to 48 and if the user upload more points, it will be redundant. #266

Closed
ehsanMa86 opened this issue May 10, 2018 · 2 comments
Closed

Curve points are limited to 48 and if the user upload more points, it will be redundant. #266

ehsanMa86 opened this issue May 10, 2018 · 2 comments
Assignees
@TongZhai
Labels
EPANET needs more info

Comments

@ehsanMa86
Copy link
Contributor

Curve points are limited to 48 and if the user upload more points, it will be redundant.

Version: EPANET MTP4r2 (master branch)
Priority: Medium
@PaulDudaRESPEC
Copy link
Collaborator

I built an example with 50 points on a curve, and it read in without a problem. Maybe the user could supply more details and an example?

TongZhai added a commit that referenced this issue Aug 7, 2019
@TongZhai
fix #266 handles arbitrary number of curve points
c309b19
@TongZhai
Copy link
Contributor

TongZhai commented Aug 7, 2019

@ehsanMa86 the fix above will allow uploading and saving of curve with arbitrary number of points, also the graphing is improved to be more efficient during loading process

@TongZhai TongZhai closed this as completed Aug 7, 2019
@TongZhai TongZhai self-assigned this Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@TongZhai TongZhai

Labels
EPANET needs more info
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@PaulDudaRESPEC @TongZhai @ehsanMa86