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.

Sign up for GitHub

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

Jump to bottom

nnx.pmap documentation seems to be for vmap #4330

Open
hrbigelow opened this issue Oct 24, 2024 · 2 comments
Open

nnx.pmap documentation seems to be for vmap #4330

hrbigelow opened this issue Oct 24, 2024 · 2 comments
Assignees
@cgarciae
Labels
Priority: P1 - soon Response within 5 business days. Resolution within 30 days. (Assignee required)

Comments

@hrbigelow
Copy link

Hi,

The source code documentation for nnx.pmap appears to be instead referring to nnx.vmap.
@cgarciae cgarciae self-assigned this Oct 25, 2024
@cgarciae cgarciae added the Priority: P1 - soon Response within 5 business days. Resolution within 30 days. (Assignee required) label Oct 25, 2024
@cgarciae
Copy link
Collaborator

This is correct. Sorry it was adapted very quickly from vmap for an experiment.
Will update soon.

@hrbigelow
Copy link
Author

No problem, thank you @cgarciae

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

@cgarciae cgarciae

Labels
Priority: P1 - soon Response within 5 business days. Resolution within 30 days. (Assignee required)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hrbigelow @cgarciae