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

Membership between singleton and lazy m-sum #1793

Closed
mforets opened this issue Nov 23, 2019 · 2 comments · Fixed by #1795
Closed

Membership between singleton and lazy m-sum #1793

mforets opened this issue Nov 23, 2019 · 2 comments · Fixed by #1795
Assignees
@mforets
Labels
extension ⬆️ Extension of an existing feature set operation

Comments

@mforets
Copy link
Member

mforets commented Nov 23, 2019

Cf. #1792 (comment)
@mforets mforets mentioned this issue Nov 23, 2019
Closed
@schillic
Copy link
Member

I think this is the same as #1562.

@schillic schillic added the duplicate 2️⃣ This issue or pull request already exists label Nov 23, 2019
@mforets
Copy link
Member Author

mforets commented Nov 26, 2019 •
edited
Loading

The method that failed can be handled more simply without ICP, as

 [1] in(::Array{Float64,1}, ::MinkowskiSum{Float64,Singleton{Float64,Array{Float64,1}},BallInf{Float64}}) at ./operators.jl:1055

can be decided by checking if the point p-s belongs to the given polytope, where p is the Array{Float64,1} and s is the singleton.

@mforets mforets reopened this Nov 26, 2019
@schillic schillic removed the duplicate 2️⃣ This issue or pull request already exists label Nov 26, 2019
@mforets mforets mentioned this issue Nov 26, 2019
Merged
@schillic schillic added extension ⬆️ Extension of an existing feature set operation labels Nov 26, 2019
mforets added a commit that referenced this issue Nov 27, 2019
@mforets
#1793 - Add membership for msum of singleton and set (#1795)
617a497 
* add membership for msum of singleton and set

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* extend to abstract singleton

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update src/LazyOperations/MinkowskiSum.jl

Co-Authored-By: Christian Schilling <[email protected]>

* Update docs/src/lib/operations.md

* change signature for documentation to work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mforets mforets

Labels
extension ⬆️ Extension of an existing feature set operation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#1793 - Add membership for msum of singleton and set JuliaReach/LazySets.jl
2 participants
@schillic @mforets