Skip to content

cp: fix: Allow use_cache w/ activation_checkpointing (1726) into r0.4.0#1760

Merged
akoumpa merged 7 commits intor0.4.0from
cherry-pick-1726-r0.4.0
Apr 17, 2026
Merged

cp: fix: Allow use_cache w/ activation_checkpointing (1726) into r0.4.0#1760
akoumpa merged 7 commits intor0.4.0from
cherry-pick-1726-r0.4.0

Conversation

@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor

beep boop [🤖]: Hi @athitten 👋,

we've cherry picked #1726 into  for you! 🚀

Please review and approve this cherry pick by your convenience!

* Allow use_cache when activation_checkpointing is True

Signed-off-by: Abhishree Thittenamane <athittenaman@cw-dfw-cs-001-login-02.cm.cluster>

* Fix lint errors

Signed-off-by: Abhishree <abhishreetm@gmail.com>

* Fix imports linting

Signed-off-by: Abhishree <abhishreetm@gmail.com>

* Add tests

Signed-off-by: Abhishree Thittenamane <athittenaman@cw-dfw-cs-001-login-02.cm.cluster>

* Add more tests

Signed-off-by: root <root@pool0-01595.cm.cluster>

---------

Signed-off-by: Abhishree Thittenamane <athittenaman@cw-dfw-cs-001-login-02.cm.cluster>
Signed-off-by: Abhishree <abhishreetm@gmail.com>
Signed-off-by: root <root@pool0-01595.cm.cluster>
Co-authored-by: Abhishree Thittenamane <athittenaman@cw-dfw-cs-001-login-02.cm.cluster>
Co-authored-by: root <root@pool0-01595.cm.cluster>
Signed-off-by: NeMo Bot <nemo-bot@nvidia.com>
@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor Author

/ok to test 75fa1ee

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 9, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@akoumpa akoumpa changed the title cp: fix: Allow use_cache when activation_checkpointing is True (1726) into r0.4.0 cp: fix: Allow use_cache w/ activation_checkpointing (1726) into r0.4.0 Apr 9, 2026
@ZhiyuLi-Nvidia
Copy link
Copy Markdown
Contributor

/ok to test 4e63025

@athitten
Copy link
Copy Markdown
Contributor

/ok to test b936976

@athitten
Copy link
Copy Markdown
Contributor

/ok to test 83beea9

Signed-off-by: Abhishree <abhishreetm@gmail.com>
@athitten
Copy link
Copy Markdown
Contributor

/ok to test b5cfffe

@athitten
Copy link
Copy Markdown
Contributor

Quick note: Removes get_submesh monkey patch from tests/unit_tests/distributed/test_parallelizer.py since the test was written against main which had #1684 and its not available in the release branch, so get_submesh doesn't exist in parallelizer.py's namespace on this branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick Run CICD Trigger Testing CICD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants