chore(processor): Improve naming conventions #1803
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AdilZouitine
changed the title
Base automatically changed from
fix/pipeline_development
to
feat/pipeline_development
August 31, 2025 18:34
Base automatically changed from
feat/pipeline_development
to
user/azouitine/2025-7-4-convert-codebase-with-pipeline
August 31, 2025 18:38
* refactor(processor): rename RobotProcessor to DataProcessorPipeline for clarity and consistency * refactor(pipeline): replace RobotProcessor with DataProcessorPipeline across multiple files for consistency
…sorPipeline as type aliases for DataProcessorPipeline (#1804) * refactor(pipeline): introduce RobotProcessorPipeline and PolicyProcessorPipeline as type aliases for DataProcessorPipeline * refactor(pipeline): replace DataProcessorPipeline with PolicyProcessorPipeline alias across multiple policy files for consistency * refactor(pipeline): update references to DataProcessorPipeline with RobotProcessorPipeline in record, replay, and teleoperate modules for improved consistency
…ename _device field (#1814) * refactor(processor): rename DeviceProcessor to DeviceProcessorStep across multiple files for consistency * refactor(processor): update device reference in DeviceProcessorStep for clarity
…rocessorStep across multiple files for consistency (#1815)
* refactor(processor): rename internal tokenizer field for clarity in TokenizerProcessor * refactor(processor): rename TokenizerProcessor to TokenizerProcessorStep for consistency across multiple files
…ormat across multiple files (#1816)
imstevenpmwork
force-pushed
the
feat/processor_improve_naming_conventions
branch
from
b3550f0 to
d428bb5
Compare
… into feat/processor_improve_naming_conventions Signed-off-by: Adil Zouitine <adilzouitinegm@gmail.com>
for more information, see https://pre-commit.ci
…_observation_to_transition for clarity (#1825) - Updated references across evaluate.py, record.py, teleoperate.py, and other files to reflect the new naming convention. - Improved readability and maintainability of the codebase.
…or clarity (#1826) - Updated references in evaluate.py, record.py, and test_dataset_utils.py to reflect the new function name. - Improved code readability and maintainability by standardizing the naming convention.
…_calibration_pose_on_trigger for clarity (#1827) - Updated method names in IOSPhone and AndroidPhone classes to improve readability and maintainability. - Ensured consistent naming convention across teleoperation modules.
…uffix for consistency (#1829) * refactor(processor): rename MapTensorToDeltaActionDict to MapTensorToDeltaActionDictStep for consistency - Updated references in init.py and gym_manipulator.py to reflect the new class name. - Improved clarity and maintainability of the codebase by standardizing naming conventions. * refactor(processor): rename MapDeltaActionToRobotAction to MapDeltaActionToRobotActionStep for consistency - Updated references in init.py, gym_manipulator.py, and delta_action_processor.py to reflect the new class name. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions. * refactor(processor): rename Torch2NumpyActionProcessor to Torch2NumpyActionProcessorStep for consistency - Updated references in init.py, gym_action_processor.py, and gym_manipulator.py to reflect the new class name. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions. * refactor(processor): rename Numpy2TorchActionProcessor to Numpy2TorchActionProcessorStep for consistency - Updated references in init.py, gym_action_processor.py, and gym_manipulator.py to reflect the new class name. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions. * refactor(processor): rename processors for consistency and clarity - Updated class names in hil_processor.py, init.py, and gym_manipulator.py to include 'Step' suffix for better consistency. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename JointVelocityProcessor and MotorCurrentProcessor for consistency - Updated class names in joint_observations_processor.py, init.py, and gym_manipulator.py to include 'Step' suffix for better consistency. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename NormalizerProcessor and UnnormalizerProcessor to include 'Step' suffix for consistency - Updated class names in normalize_processor.py and various processor files to reflect the new naming convention. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename VanillaObservationProcessor to VanillaObservationProcessorStep for consistency - Updated references in observation_processor.py, init.py, gym_manipulator.py, and test_observation_processor.py to reflect the new class name. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename NormalizerProcessor and UnnormalizerProcessor to NormalizerProcessorStep and UnnormalizerProcessorStep for consistency - Updated references in migrate_policy_normalization.py to reflect the new class names. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename RenameProcessor to RenameProcessorStep for consistency - Updated references across multiple processor files and tests to reflect the new class name. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(processor): rename ToBatchProcessor classes to include 'Step' suffix for consistency - Updated class names in batch_processor.py to reflect the new naming convention. - Enhanced clarity and maintainability of the codebase by standardizing naming conventions across processor classes. * refactor(tests): update config path in test_save_and_load_pretrained to reflect RenameProcessorStep naming - Adjusted the configuration file path in the test to match the updated class name "TestRenameProcessorStep". - Ensured consistency in naming conventions across test files.
AdilZouitine
added
refactor
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.