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

fix: fix 0 stride in CG pool management #291

Merged
merged 1 commit into from
Feb 17, 2023
Merged

Conversation

pommedeterresautee
Copy link
Member

fix #286

@pommedeterresautee
Copy link
Member Author

test pass

=============================== warnings summary ===============================
test/test_model_optimization.py: 1 warning
test/test_torchdynamo.py: 79 warnings
  /home/geantvert/.local/share/virtualenvs/kernl/lib/python3.9/site-packages/torch/cuda/graphs.py:82: UserWarning: The CUDA Graph is empty. This ususally means that the graph was attempted to be captured on wrong device or stream. (Triggered internally at ../aten/src/ATen/cuda/CUDAGraph.cpp:191.)
    super(CUDAGraph, self).capture_end()

test/debugger/test_memory.py::test_load_is_in_different_memory
  /mnt/workspace/kernl/test/debugger/test_memory.py:58: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
    assert t.storage().data_ptr() != a.storage().data_ptr()

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
========== 2880 passed, 3 skipped, 81 warnings in 8679.71s (2:24:39) ===========

@pommedeterresautee pommedeterresautee merged commit 4e4cf69 into main Feb 17, 2023
@pommedeterresautee pommedeterresautee deleted the fix/0_stride branch February 17, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix hurrah, bug fixed!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

I run the bert e2e example, if batch is not 1, I get an error!!!
2 participants