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

Remove local variables with same name when adding system parameters #2225

Merged
merged 1 commit into from
May 27, 2024
Merged

Remove local variables with same name when adding system parameters #2225

merged 1 commit into from
May 27, 2024

Conversation

robbr48
Copy link
Contributor

@robbr48 robbr48 commented May 27, 2024

If they are not removed, the following problem will occur:

>> y=3
Assigning scalar y with 3
>> adpa y 4
>> y
3

The value in the system parameters widget, however, was actually 4.

@robbr48 robbr48 added the gui:scripting HCOM, Python or Embedded scripting label May 27, 2024
@robbr48 robbr48 added this to the 2.22.0 milestone May 27, 2024
@robbr48 robbr48 self-assigned this May 27, 2024
@robbr48 robbr48 merged commit 1884743 into Hopsan:master May 27, 2024
3 checks passed
@robbr48 robbr48 deleted the adpa_remove_local_vars branch September 30, 2024 11:51
@robbr48
Copy link
Contributor Author

robbr48 commented Sep 30, 2024

Resolves #2115.

@robbr48 robbr48 mentioned this pull request Sep 30, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@robbr48 robbr48

Labels
gui:scripting HCOM, Python or Embedded scripting
Projects
None yet
Milestone
2.22.0
Development

Successfully merging this pull request may close these issues.
1 participant
@robbr48