fiftyone icon indicating copy to clipboard operation
fiftyone copied to clipboard

fix resolve input not debounce

Open imanjra opened this issue 11 months ago • 2 comments

What changes are proposed in this pull request?

Typo in PR https://github.com/voxel51/fiftyone/pull/4152 regressed debounced resolution of operator input causing operator prompt to re-resolve input in every change and potentially degrading performance. Changes in this PR fixes the issue to ensure input schema is not resolved for every change.

How is this patch tested? If it is not, please explain why.

Using a dynamic operator.

Release Notes

Is this a user-facing change that should be mentioned in the release notes?

  • [ ] No. You can skip the rest of this section.
  • [x] Yes. Give a description of this change to be included in the release notes for FiftyOne users.

fix the dynamic operator's input schema getting resolved for every change

What areas of FiftyOne does this PR affect?

  • [x] App: FiftyOne application changes
  • [ ] Build: Build and test infrastructure changes
  • [ ] Core: Core fiftyone Python library changes
  • [ ] Documentation: FiftyOne documentation changes
  • [ ] Other

imanjra avatar Mar 26 '24 13:03 imanjra