Skip to content

Improve tests for scalar optimizers#39

Merged
skyw merged 7 commits into
NVIDIA-NeMo:mainfrom
mkhona-nvidia:mkhona/soap_laprop
Oct 6, 2025
Merged

Improve tests for scalar optimizers#39
skyw merged 7 commits into
NVIDIA-NeMo:mainfrom
mkhona-nvidia:mkhona/soap_laprop

Conversation

@mkhona-nvidia

@mkhona-nvidia mkhona-nvidia commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Added flags for scalar optimizer test and improved device handling

@copy-pr-bot

copy-pr-bot Bot commented Oct 3, 2025

Copy link
Copy Markdown

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.

Comment thread emerging_optimizers/scalar_optimizers/laprop.py Outdated
Comment thread tests/test_scalar_optimizers.py Outdated
Comment thread tests/test_scalar_optimizers.py Outdated
Comment thread tests/test_scalar_optimizers.py Outdated
Comment thread tests/test_scalar_optimizers.py Outdated
Comment thread tests/test_scalar_optimizers.py Outdated
Comment thread tests/test_scalar_optimizers.py
Comment thread tests/test_scalar_optimizers.py Outdated
@skyw

skyw commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

LGTM.
@mkhona-nvidia resolve the conflict and trigger the CI please.

@mkhona-nvidia mkhona-nvidia self-assigned this Oct 6, 2025
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
Signed-off-by: mikail <mkhona@nvidia.com>
@mkhona-nvidia

Copy link
Copy Markdown
Contributor Author

/ok to test b7fd643

@skyw skyw merged commit e1a8def into NVIDIA-NeMo:main Oct 6, 2025
12 checks passed
mkhona-nvidia added a commit to mkhona-nvidia/Emerging-Optimizers that referenced this pull request Oct 7, 2025
* test on both CPU and GPU

Signed-off-by: mikail <mkhona@nvidia.com>
mkhona-nvidia added a commit to mkhona-nvidia/Emerging-Optimizers that referenced this pull request Oct 7, 2025
* test on both CPU and GPU

Signed-off-by: mikail <mkhona@nvidia.com>
@mkhona-nvidia mkhona-nvidia deleted the mkhona/soap_laprop branch October 15, 2025 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants