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.

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

create record_dict on data device in lif_torch.py #13

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

kismde
Copy link

@kismde kismde commented Jun 30, 2023

No description provided.

@DylanMuir DylanMuir added the enhancement New feature or request label Jul 4, 2023
@ugurcancakal
Copy link
Contributor

Hi @wangkgege,

Thanks for the suggestion! Could you please tell us a bit more?

  • Why do you need the record dictionary to be allocated on GPU?
    • For example do you use any of the items in your training pipeline?
    • Do you need all of them there?
    • Would it make sense to you to choose only one of them like vmem or isyn to be allocated on GPU?

I try to understand better because it might be an expensive decision in terms of memory usage to allocate the whole record dict in GPU.

Best,
Ugurcan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants