Skip to content

Small updates from Caltech CDS 110 #995

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

Closed
wants to merge 2 commits into from

Conversation

murrayrm
Copy link
Member

This (draft) PR contains some small fixes that I am making as I teach CDS 110 at Caltech and notice things that can be improved in v0.10.0. Depending on timing and importance, we can decide to include some of all of these into the next patch release (0.10.1).

@coveralls
Copy link

Coverage Status

coverage: 94.504%. remained the same
when pulling 4905855 on murrayrm:cds110_fixes-16Apr2024
into 1bb0a46 on python-control:main.

Copy link
Contributor

@bnavigator bnavigator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The two commits look good to me. Why don't you PR the small fixes one by one and we can include them in to the next release as they come?

@murrayrm
Copy link
Member Author

Pulling this PR in favor of smaller ones focused on specific issues, as suggested by @bnavigator.

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.

3 participants