Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Kernel for calculating additional fields that require neigbor traversal #416

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

lks1248
Copy link
Collaborator

@lks1248 lks1248 commented Oct 8, 2023

initial implementation, including markRamp calculation. I would be happy to take some feedback on the general layout etc. I will test the functionality and correctness before moving the PR to ready for merging.

@lks1248 lks1248 requested a review from sekelle October 8, 2023 12:15
@lks1248 lks1248 marked this pull request as ready for review October 14, 2023 09:47
@lks1248
Copy link
Collaborator Author

lks1248 commented Oct 14, 2023

The calculations work fine, I am just wondering if I could deallocate markRamp most of the time, it seems to be allocated by default.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants