Skip to content

Fixes for changes in how ttg::Buffer takes pointers - #43

Merged
nileshchat merged 1 commit into
nileshchat:mainfrom
devreal:ttg-buffer-changes
Jan 28, 2025
Merged

Fixes for changes in how ttg::Buffer takes pointers#43
nileshchat merged 1 commit into
nileshchat:mainfrom
devreal:ttg-buffer-changes

Conversation

@devreal

@devreal devreal commented Jan 27, 2025

Copy link
Copy Markdown
Collaborator

All pointers must now be managed so that they don't disappear silently. This needs some adjustment in how we allocate the Domain and gaussian arrays.

In preparation for TESSEorg/ttg#301

@devreal
devreal requested a review from nileshchat January 27, 2025 22:39

@nileshchat nileshchat left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CMake Error at CMakeLists.txt:14 (enable_language):
  The CMAKE_CUDA_COMPILER:

    /usr/local/cuda-12.6/bin/nvcc

  is not a full path to an existing compiler tool.

Is this perhaps cuda 12.0 vs cuda 12.6 ?

All pointers must now be managed so that they don't disappear
silently. This needs some adjustment in how we allocate
the Domain and gaussian arrays.

Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
@devreal
devreal force-pushed the ttg-buffer-changes branch from e2e3254 to a608c30 Compare January 28, 2025 01:55
@devreal

devreal commented Jan 28, 2025

Copy link
Copy Markdown
Collaborator Author

That was fixed earlier in #44

@nileshchat
nileshchat merged commit b91a28c into nileshchat:main Jan 28, 2025
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