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

Contains() is incorrect #13

Closed
io7m opened this Issue May 8, 2017 · 0 comments

Comments

Projects
None yet
1 participant
@io7m
Owner

io7m commented May 8, 2017

See #12

The problem is the same!

@io7m io7m added the bug label May 8, 2017

@io7m io7m added this to the 1.1.0 milestone May 8, 2017

@io7m io7m self-assigned this May 8, 2017

@io7m io7m closed this in 0b54c09 May 8, 2017

io7m added a commit that referenced this issue May 9, 2017

Fix overlaps() again
Correct overlaps() for areas/volumes with a zero width/height. This
treats all areas or volumes as having a width, height, or depth of
at least 1.

Fix #13
Fix #14

io7m added a commit that referenced this issue May 9, 2017

Merge branch 'release/1.1.0'
Release: com.io7m.jregions 1.1.0
Code new: Add splitAlongXY method for dividing areas into quadrants.
Code new: Stop using terms such as "horizontally", "vertically", "width", "height", etc. Deprecate methods and add replacements. (tickets: #11)
Code new: Add volume types. (tickets: #9)
Code fix: Fix overlaps() for areas and volumes. (tickets: #12, #14)
Code fix: Fix contains() for areas and volumes. (tickets: #13)

io7m added a commit that referenced this issue May 9, 2017

Merge tag 'com.io7m.jregions-1.1.0' into develop
Release: com.io7m.jregions 1.1.0

Code new: Add splitAlongXY method for dividing areas into quadrants.
Code new: Stop using terms such as "horizontally", "vertically", "width", "height", etc. Deprecate methods and add replacements. (tickets: #11)
Code new: Add volume types. (tickets: #9)
Code fix: Fix overlaps() for areas and volumes. (tickets: #12, #14)
Code fix: Fix contains() for areas and volumes. (tickets: #13)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment