# Unnecessary VTK API change

**URL:** <https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929>\
**Category:** Development\
**Created:** [November 23, 2022, 2:33am UTC](https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929 "2022-11-23T02:33:43Z")\
**Posts on this page:** 3\
**Page:** 2

<div class="post-metadata">

**Author:** ![banesullivan](https://discourse.vtk.org/user_avatar/discourse.vtk.org/banesullivan/32/7143_2.png) [@banesullivan](https://discourse.vtk.org/u/banesullivan)\
**Post date:** [November 29, 2022, 6:42pm UTC](https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929/21 "2022-11-29T18:42:04Z")

</div>

> [@mwestphal](#):
>
> One of note from pyvista (@banesullivan ):
> 
> > We wouldn’t blame you for being ruthlessly annoyed by the continual deprecation churn from Vtk. We kinda are. Like, seriously. Vtk, can you just slow down that moving API target a little there? face\_exhaling

I want to clarify that I wasn’t the one who said that… 😄 Originally posted here: [vtkThreshold::ThresholdByUpper was deprecated for VTK 9.1 and will be removed in a future version. · Issue #2850 · pyvista/pyvista · GitHub](https://github.com/pyvista/pyvista/issues/2850)

> [@mwestphal](#):
>
> That being said, this should have been caught much earlier imo in order to avoid forcing VTK users to change their use of VTK.

Agreed!

---

<div class="post-metadata">

**Author:** ![mwestphal](https://discourse.vtk.org/user_avatar/discourse.vtk.org/mwestphal/32/19_2.png) [@mwestphal](https://discourse.vtk.org/u/mwestphal)\
**Post date:** [November 30, 2022, 7:33am UTC](https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929/22 "2022-11-30T07:33:05Z")

</div>

> [@banesullivan](#):
>
> I want to clarify that I wasn’t the one who said that… 😄 Originally posted here:

Indeed unclear, I’ve edited my post to reflect that.

---

<div class="post-metadata">

**Author:** ![lassoan](https://discourse.vtk.org/user_avatar/discourse.vtk.org/lassoan/32/50_2.png) [@lassoan](https://discourse.vtk.org/u/lassoan)\
**Post date:** [November 30, 2022, 1:37pm UTC](https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929/23 "2022-11-30T13:37:05Z")

</div>

> [@mwestphal](#):
>
> @lassoan , it looks like @jcfr and yourself fixed that in slicer in June, what made you open this thread now and not at the time ? I’m just trying to understand what I did wrong here (apart misdjuging the usefulness of an API).

We changed most of the usages of the API, but there were some in Python that slipped our testing and found them months later, which forced us to create an urgent patch release.

The VTK API change looked small enough that we didn’t realize its actual impact at the time, but when it caused us this extra complication then we realized that it could cause trouble in many other VTK projects.

[Previous page](https://discourse.vtk.org/t/unnecessary-vtk-api-change/9929.md?page=1)