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

[XRI3] Remove obsolete methods (with XRI 3.0 Updates) #531

Closed
david-c-kline opened this issue Oct 25, 2023 · 6 comments
Closed

[XRI3] Remove obsolete methods (with XRI 3.0 Updates) #531

david-c-kline opened this issue Oct 25, 2023 · 6 comments
Assignees
Labels
Package: Core The Project core package is impacted by this issue. Package: Input The Project input package is impacted by this issue. Package: Spatial Manipulation The Project spatial manipulation package is impacted by this issue. Package: UX Core The Project ux core package is impacted by this issue. Priority: High The priority of the issue is high. Type: Feature Request A request for a new feature that can be included with the next minor version release.

Comments

@david-c-kline
Copy link
Contributor

It was originally intended that obsolete methods be removed prior to GA. Unfortunately, a few packages still have methods that should not be used.

NOTE: Removing these methods will require a major version update for each impacted package.

org.mixedrealitytoolkit.core\Utilities\Extensions\HandednessExtensions.cs:44: [Obsolete("Use flags instead (e.g. Handedness.Right | Handedness.Left)")]
org.mixedrealitytoolkit.core\Utilities\Extensions\HandednessExtensions.cs:56: [Obsolete("Use flags instead (e.g. Handedness.Right | Handedness.Left)")]
org.mixedrealitytoolkit.core\Utilities\Extensions\HandednessExtensions.cs:68: [Obsolete("Use flags instead (e.g. Handedness.Right | Handedness.Left)")]
org.mixedrealitytoolkit.core\Utilities\HandsUtils.cs:22: [Obsolete("Deprecated, please use XRSubsystemHelpers.HandsAggregator")]
org.mixedrealitytoolkit.core\Utilities\PlayspaceUtilities.cs:44: [ObsoleteAttribute("For transforming poses into world space, use XROrigin.CameraFloorOffsetObject.transform.")]
org.mixedrealitytoolkit.core\Utilities\XRSubsystemHelpers.cs:55: [Obsolete("GetAllRunningSubsystems now internally caches for you; this separate NonAlloc method is no longer necessary.")]
org.mixedrealitytoolkit.input\Controllers\ArticulatedHandController.cs:50: [Obsolete("Deprecated, please use XRSubsystemHelpers.HandsAggregator instead.")]
org.mixedrealitytoolkit.input\Interactors\Ray\MRTKRayInteractor.cs:52: [Obsolete("Deprecated, please use XRSubsystemHelpers.HandsAggregator instead.")]
org.mixedrealitytoolkit.input\Simulation\Devices\ControllerSimulationSettings.cs:61: [Obsolete("This property is deprecated, use ToggleSecondaryHandshapes instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:76: [Obsolete("Please use the GetNeutralHandshape(handNode) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:84: [Obsolete("Please use the SetNeutralHandshape(handNode, handshapeId) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:94: [Obsolete("Please use the GetSelectionHandshape(handNode) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:102: [Obsolete("Please use the SetSelectionHandshape(handNode, handshapeId) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:469: [Obsolete("Please use the GetNeutralHandshape(handNode) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:477: [Obsolete("Please use the SetNeutralHandshape(handNode, handshapeId) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:487: [Obsolete("Please use the GetSelectionHandshape(handNode) instead.")]
org.mixedrealitytoolkit.input\Subsystems\SyntheticHands\SyntheticHandsSubsystem.cs:495: [Obsolete("Please use the SetSelectionHandshape(handNode, handshapeId) instead.")]
org.mixedrealitytoolkit.input\Utilities\FollowJoint.cs:42: [Obsolete("Please change the Hand value on the jointPoseSource instead")]
org.mixedrealitytoolkit.input\Utilities\FollowJoint.cs:52: [Obsolete("Please change the Joint value on the jointPoseSource instead")]
org.mixedrealitytoolkit.input\Utilities\PoseSource\FallbackCompositePoseSource.cs:23: [Obsolete, HideInInspector]
org.mixedrealitytoolkit.input\Utilities\PoseSource\FallbackCompositePoseSource.cs:50: [Obsolete]
org.mixedrealitytoolkit.input\Utilities\PoseSource\HandBasedPoseSource.cs:18: [Obsolete("Deprecated, please use XRSubsystemHelpers.HandsAggregator instead.")]
org.mixedrealitytoolkit.input\Visualizers\ControllerVisualizer\ControllerVisualizer.cs:50: [Obsolete("Deprecated, please use XRSubsystemHelpers.HandsAggregator instead.")]
org.mixedrealitytoolkit.spatialmanipulation\Constraints\ConstraintManager.cs:140: [Obsolete("Use Setup instead for first-time initialization, and OnManipulationStarted for subsequent manipulation.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:19: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:25: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:31: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:37: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:47: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:61: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:75: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:89: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:103: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:117: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:131: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:148: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:158: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:165: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:171: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:178: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:199: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:235: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\Dialog.cs:278: [Obsolete("Legacy Dialog is deprecated. Please migrate to the new Dialog. See uxcore/LegacyDialog/README.md")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:21: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:30: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:40: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:46: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:50: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:57: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:63: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:74: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:88: [Obsolete("Use SetLabel or SetButtonContext instead.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:95: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButton.cs:110: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonContext.cs:20: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonContext.cs:26: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonContext.cs:37: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonContext.cs:45: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonHelpers.cs:18: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonHelpers.cs:24: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonHelpers.cs:30: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonHelpers.cs:36: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonTypes.cs:18: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogButtonTypes.cs:73: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:19: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:28: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:46: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:65: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:85: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:105: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:111: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:117: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:123: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:129: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:135: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogProperty.cs:141: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:23: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:33: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:47: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:55: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:59: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogShell.cs:116: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\LegacyDialog\DialogState.cs:18: [Obsolete("This legacy dialog system has been deprecated. Please migrate to the new dialog system, see MixedReality.Toolkit.UX.DialogPool for more details.")]
org.mixedrealitytoolkit.uxcore\Slider\Slider.cs:143: [Obsolete("Use Value instead")]

@david-c-kline david-c-kline added Type: Feature Request A request for a new feature that can be included with the next minor version release. Needs: Triage Needs to be triaged. labels Oct 25, 2023
@AMollis AMollis added Package: Core The Project core package is impacted by this issue. Package: Input The Project input package is impacted by this issue. Package: Spatial Manipulation The Project spatial manipulation package is impacted by this issue. Package: UX Core The Project ux core package is impacted by this issue. and removed Needs: Triage Needs to be triaged. labels Oct 26, 2023
@AMollis
Copy link
Contributor

AMollis commented Oct 26, 2023

Maintainers discussed, and agreed this should happen with next major release.

This would impact the following packages:

  • Core
  • UXCore
  • Input
  • SpatialManipulation

@AMollis AMollis added the Priority: High The priority of the issue is high. label Jan 2, 2024
@shaynie
Copy link
Contributor

shaynie commented Apr 10, 2024

Reevaluate with XRI 3.0 upgrade

@AMollis AMollis changed the title Remove obsolete methods Remove obsolete methods (with XRI 3.0 Updates) Apr 24, 2024
@AMollis
Copy link
Contributor

AMollis commented Apr 24, 2024

@ms-RistoRK
Copy link
Contributor

@ms-RistoRK can you remove the Obsolete stuff in the https://github.com/MixedRealityToolkit/MixedRealityToolkit-Unity/tree/feature/XRI3 branch?

Sure, I'll create a Task in the corresponding User Story to address the issue as part of the XRI 3 migration.

@whebertML
Copy link
Contributor

@ms-RistoRK @AMollis Was slow to review the latest XRI3 commit, #743, but notice that we are starting to add new [Obsolete] attributes to some classes and methods. Was thinking should the next steps be to remove previous [Obsolete] methods/classes before we start adding too many new ones, since we are only removing the previous/legacy obsolete methods and will be keeping the new XRI3-based ones, correct? Would probably be easier to remove the legacy ones before we add too many new ones, so as to avoid confusion. Happy to help with that if needed.

@ms-RistoRK
Copy link
Contributor

PR for this issue is ready for review: #774

@AMollis AMollis closed this as completed Jul 25, 2024
@AMollis AMollis changed the title Remove obsolete methods (with XRI 3.0 Updates) [XRI3] Remove obsolete methods (with XRI 3.0 Updates) Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: Core The Project core package is impacted by this issue. Package: Input The Project input package is impacted by this issue. Package: Spatial Manipulation The Project spatial manipulation package is impacted by this issue. Package: UX Core The Project ux core package is impacted by this issue. Priority: High The priority of the issue is high. Type: Feature Request A request for a new feature that can be included with the next minor version release.
Projects
None yet
Development

No branches or pull requests

5 participants