• Mayank Mittal's avatar
    Fixes order of logging metrics and sampling commands in command manager (#1352) · fc6042f3
    Mayank Mittal authored
    # Description
    
    In the command manager, we were logging the metrics after resampling the
    commands. This leads to incorrect logging of metrics when inside the
    `resample` call, the metrics tensors get reset. This MR fixes the order
    to make sure the bug doesn't happen.
    
    ## Type of change
    
    - Bug fix (non-breaking change which fixes an issue)
    
    ## Checklist
    
    - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
    `./isaaclab.sh --format`
    - [x] I have made corresponding changes to the documentation
    - [x] My changes generate no new warnings
    - [ ] I have added tests that prove my fix is effective or that my
    feature works
    - [ ] I have updated the changelog and the corresponding version in the
    extension's `config/extension.toml` file
    - [x] I have added my name to the `CONTRIBUTORS.md` or my name already
    exists there
    
    ---------
    Signed-off-by: 's avatarKelly Guo <kellyg@nvidia.com>
    Co-authored-by: 's avatarKelly Guo <kellyg@nvidia.com>
    Co-authored-by: 's avatarKelly Guo <kellyguo123@hotmail.com>
    fc6042f3
Name
Last commit
Last update
..
omni.isaac.lab Loading commit data...
omni.isaac.lab_assets Loading commit data...
omni.isaac.lab_tasks Loading commit data...