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.

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

increase ldwork for dico=='D' #166

Merged
merged 1 commit into from
Jan 19, 2022
Merged

increase ldwork for dico=='D' #166

merged 1 commit into from
Jan 19, 2022

Conversation

artpelling
Copy link

Fixes #165.

@bnavigator
Copy link
Collaborator

Okay thanks. If I am not mistaken this also makes it large enough for all other combinations of job, fact and dico.

@bnavigator
Copy link
Collaborator

Would you be willing to create some unit tests with appropriate parameter combinations and array sizes?

@artpelling artpelling mentioned this pull request Jan 19, 2022
3 tasks
@artpelling
Copy link
Author

Would you be willing to create some unit tests with appropriate parameter combinations and array sizes?

Unfortunately, I do not have the time at the moment. I looked into it, but it is not immediately clear to me what would be needed for this. Sorry

@bnavigator bnavigator merged commit 10647cd into python-control:master Jan 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect initialization of LDWORK in sb03md
2 participants