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

No records insert to m_ScenarioManagementUpdatedFields when it is needed. #42

Closed
wuwwen opened this issue Aug 22, 2024 · 1 comment · Fixed by #76
Closed

No records insert to m_ScenarioManagementUpdatedFields when it is needed. #42

wuwwen opened this issue Aug 22, 2024 · 1 comment · Fixed by #76
Labels
bug Something isn't working

Comments

@wuwwen
Copy link
Collaborator

wuwwen commented Aug 22, 2024

if the alternative scenario (the new created with new altid) record's attribute value is changed in the MIKE+ GUI, a new record of the format "Pipe_muid; Pipe_altid; Pipe_attribute" gets inserted into the "m_ScenarioManagementUpdatedFields" table in the sqlite database representing the current model.
I didn't see this happen if I modified the new pipe record's attribute value using the data_access.set_values() method via mikepluspy module.
@wuwwen
Copy link
Collaborator Author

wuwwen commented Aug 23, 2024

This issue reported by Vojtech Synacek who is in Prague

wuwwen added a commit that referenced this issue Aug 23, 2024
@wuwwen
To fix issue #42: No records insert to m_ScenarioManagementUpdatedFie…
9e99316 
…lds when it is needed.
@ryan-kipawa ryan-kipawa added the bug Something isn't working label Sep 30, 2024
@ryan-kipawa ryan-kipawa mentioned this issue Oct 4, 2024
Closed
3 tasks
@wuwwen wuwwen closed this as completed Nov 5, 2024
@wuwwen wuwwen mentioned this issue Nov 26, 2024
Merged
ryan-kipawa added a commit that referenced this issue Nov 28, 2024
@wuwwen @ryan-kipawa
To support release 2025 (#78)
9576e20 
* Update assembly version for MIKE 2025

* To fix issue #42: No records insert to m_ScenarioManagementUpdatedFields when it is needed.

* To support 2025 version

* Add full test action for release

* Update test action secret

* Update change log

---------

Co-authored-by: Ryan Walter Murray <rywm@dhigroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix the data update records missing for sub scenrio. DHI/mikepluspy
2 participants
@wuwwen @ryan-kipawa