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

one_hot_encoding uses too much gpu memory #30

Closed
sklam opened this issue Jul 18, 2017 · 3 comments
Closed

one_hot_encoding uses too much gpu memory #30

sklam opened this issue Jul 18, 2017 · 3 comments

Comments

@sklam
Copy link
Contributor

sklam commented Jul 18, 2017

A user reported that one_hot_encoding with float64 uses too much memory. It was observed that float32 uses the correct amount of memory.

@sklam sklam added the bug Something isn't working label Jul 18, 2017
@sklam
Copy link
Contributor Author

sklam commented Jul 19, 2017

Upon initial investigation using nvidia-smi, I don't find any evidence that is one_hot_encoding overallocating.

@sklam
Copy link
Contributor Author

sklam commented Jul 20, 2017

This may be a issue with auto printing in ipython notebook (i.e. last line non-assignment line in cell). If the dataframe df appears on the last line of a cell:

In[1]: df

It will be printed and also stored in the Out dictionary. So, a later del df will not fully remove the dataframe from memory because Out has a reference to it. A del Out[1] will work here but it is tedious to deal with that in every cell. Another workaround is to explicitly print(df) or str(df) to avoid Out acquiring a reference on any large object.

@sklam sklam removed the bug Something isn't working label Jul 20, 2017
@sklam
Copy link
Contributor Author

sklam commented Aug 3, 2017

The issue is most likely due to how gpu dataframes are used in the notebook. Closing.

@sklam sklam closed this as completed Aug 3, 2017
sklam added a commit to sklam/pygdf that referenced this issue Jun 11, 2018
OlivierNV pushed a commit that referenced this issue Oct 15, 2019
raydouglass pushed a commit that referenced this issue Nov 7, 2023
This PR removes --no-deps flags in jobs where nothing has been installed yet.
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

No branches or pull requests

1 participant