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 RubberBand2D interactor style #872

Merged

Conversation

GuillaumeFavelier
Copy link
Contributor

@GuillaumeFavelier GuillaumeFavelier commented Aug 19, 2020

This PR adds the 'RubberBand2D' interactor style.

  • Update testing

Closes #871

@GuillaumeFavelier GuillaumeFavelier added the enhancement Changes that enhance the library label Aug 19, 2020
@GuillaumeFavelier GuillaumeFavelier self-assigned this Aug 19, 2020
@GuillaumeFavelier
Copy link
Contributor Author

GuillaumeFavelier commented Aug 19, 2020

Should I force-enable camera parallel projection mode for the self.renderers?

            for renderer in self.plotter.renderers:
                renderer.enable_parallel_projection()

@GuillaumeFavelier GuillaumeFavelier changed the title 🚧 Add RubberBand2D interactor style Add RubberBand2D interactor style Aug 19, 2020
@GuillaumeFavelier
Copy link
Contributor Author

This is ready for reviews @pyvista/developers

Copy link
Contributor

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

I think that the docs are clear enough that parallel projection mode should be enabled if you want zooming to work with this interactor

@GuillaumeFavelier GuillaumeFavelier merged commit 8b6ca75 into pyvista:master Aug 20, 2020
@GuillaumeFavelier GuillaumeFavelier deleted the feat/rubber_band_2d branch August 20, 2020 08:39
akaszynski added a commit that referenced this pull request Sep 2, 2020
* 🌐 Internationalization and localization.

* 🏷️ add tag to translate title

* Patch 6 (#3)

* typo fix (#864)

* ✏️ fix format of python code (#866)

* fix format of python code

We need 1 blank line before python code.

* fix format of python code

* fix format of python code

* fix format of python code

* fix format of python code

* Control points merging in the glyph filter (#870)

* Add clean paramter to the glyph filter

* Use tolerance parameter

* Set default tolerance=None

* Add RubberBand2D interactor style (#872)

* Add RubberBand2D interactor style

* Update testing

* Use format instead of fstring

* Enable smooth shading for textures (#865)

* ✏️ Fix typos. (#875)

* ✏️ Fix typos. (#876)

* ✏️ Fix typos.

* ✏️ Fix typos.

* 🌐 Internationalization and localization.

* 🏷️ add tag to translate title

Co-authored-by: Guillaume Favelier <guillaume.favelier@gmail.com>
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>

* Update docs/index.rst

Co-authored-by: Alex Kaszynski <akascap@gmail.com>

* small fix by review

Co-authored-by: Guillaume Favelier <guillaume.favelier@gmail.com>
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>
Co-authored-by: Alex Kaszynski <akascap@gmail.com>
@akaszynski akaszynski mentioned this pull request Sep 8, 2020
akaszynski added a commit that referenced this pull request Jan 22, 2021
* Patch 6 (#3)

* typo fix (#864)

* ✏️ fix format of python code (#866)

* fix format of python code

We need 1 blank line before python code.

* fix format of python code

* fix format of python code

* fix format of python code

* fix format of python code

* Control points merging in the glyph filter (#870)

* Add clean paramter to the glyph filter

* Use tolerance parameter

* Set default tolerance=None

* Add RubberBand2D interactor style (#872)

* Add RubberBand2D interactor style

* Update testing

* Use format instead of fstring

* Enable smooth shading for textures (#865)

* ✏️ Fix typos. (#875)

* ✏️ Fix typos. (#876)

* ✏️ Fix typos.

* ✏️ Fix typos.

* 🌐 Internationalization and localization.

* 🏷️ add tag to translate title

Co-authored-by: Guillaume Favelier <guillaume.favelier@gmail.com>
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>

* Patch 6 (#4)

* typo fix (#864)

* ✏️ fix format of python code (#866)

* fix format of python code

We need 1 blank line before python code.

* fix format of python code

* fix format of python code

* fix format of python code

* fix format of python code

* Control points merging in the glyph filter (#870)

* Add clean paramter to the glyph filter

* Use tolerance parameter

* Set default tolerance=None

* Add RubberBand2D interactor style (#872)

* Add RubberBand2D interactor style

* Update testing

* Use format instead of fstring

* Enable smooth shading for textures (#865)

* ✏️ Fix typos. (#875)

* ✏️ Fix typos. (#876)

* ✏️ Fix typos.

* ✏️ Fix typos.

* 🌐 Internationalization and localization.

* 🏷️ add tag to translate title

* Update docs/index.rst

Co-authored-by: Alex Kaszynski <akascap@gmail.com>

* small fix by review

Co-authored-by: Guillaume Favelier <guillaume.favelier@gmail.com>
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>
Co-authored-by: Alex Kaszynski <akascap@gmail.com>

* 📷 Add test camera zoom and up

* 📷 Add camera zoom and up

* ♻️ Add Camera class

* ♻️ Add docstring

* ♻️ Add Camera class

* ♻️ Add Camera class

* ♻️ Fix arg name

* ♻️ Update Pythonic

* Fix error

* Fix error

* Fix error

* Change Camera class

inherit from vtkCamera

* Clean import

* Move the Camera class to its own file.

* Improve coverage with unit testing specific to the camera class

Co-authored-by: Guillaume Favelier <guillaume.favelier@gmail.com>
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>
Co-authored-by: Alex Kaszynski <akascap@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Changes that enhance the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RubberBand2D interactor style
3 participants