Skip to content

scalar mul for tf #583

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
Closed
wants to merge 1 commit into from
Closed

scalar mul for tf #583

wants to merge 1 commit into from

Conversation

@baggepinnen
Copy link
Member

@baggepinnen baggepinnen commented Dec 22, 2021

No description provided.

@JuliaControlBot
Copy link

JuliaControlBot commented Dec 22, 2021

Something failed when generating plots. See the log at https://github.com/JuliaControl/ControlExamplePlots.jl/actions/runs/1610229633?check_suite_focus=true for more details.

albheim
albheim reviewed Dec 22, 2021
View reviewed changes
Copy link
Member

@albheim albheim left a comment

Choose a reason for hiding this comment

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

We had some discussion about this is #166 where it seemed like it was maybe better to not allow this. I think both could be okay, just putting it here in case you had missed that discussion.

@baggepinnen baggepinnen mentioned this pull request Jan 3, 2022
Merged
@baggepinnen baggepinnen closed this Jan 5, 2022
@baggepinnen baggepinnen deleted the fb/tfmul branch January 6, 2022 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@albheim albheim albheim left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@baggepinnen @JuliaControlBot @albheim