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

add ignoreunits option to connect calls within @defcomposite #701

Open
corakingdon opened this issue Apr 11, 2020 · 0 comments
Open

add ignoreunits option to connect calls within @defcomposite #701

corakingdon opened this issue Apr 11, 2020 · 0 comments
Assignees
Milestone

Comments

@corakingdon
Copy link
Collaborator

We should probably allow the following:

@defcomp A begin
    p = Parameter(unit="dollars")
end

@defcomp B begin
    v = Variable(unit="dol")
end

@defcomposite C begin
    Component(A)
    Component(B)
    connect(A.p, B.v, ignoreunits=true)
end
@corakingdon corakingdon added this to the v0.10.0 milestone Apr 11, 2020
@lrennels lrennels self-assigned this Apr 27, 2020
@corakingdon corakingdon modified the milestones: v0.10.0, v1.1.0 May 5, 2020
@davidanthoff davidanthoff modified the milestones: v1.1.2, Backlog Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants