-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
64bit edges not reflected in the headers #41
Comments
thanks, will integrate it with the next version. |
Would it be possible/reasonable to define a typedef such as |
Hey Mark, I am currently looking into this. Could you explain a little bit more what you need? Maybe I am able to integrate it in the current sprint. |
Hi, I still meet this problem with the latest master branch. The type in |
Hi, I am happy if you issue a pull request. |
@schulzchristian I feel it's complicated. The signatures in https://github.com/KaHIP/KaHIP/blob/2158c5b6c07ae1b5755bcda6ea7adbca1a3618a2/interface/kaHIP_interface.cpp are all int. Is it safe to replace them with |
@VoVAllen nice that you are looking at this. I ran out of time to pursue it further... |
I will look into it next week. Best |
Along these lines if a typedef is added would it be possible to also add a function to query the size? Something like
This is extremely useful for ABI purposes as it means given |
Hi @FreddieWitherden, I am happy to add this through a pull request. |
Perfect! I will work on something over the next few days and submit for feedback. |
First attempt at this is in #148 |
If I compile with 64bit edges, I'd expect a modified of the
kaHIP_interface.h
file to reflect this, but it only seems to be unaffected. The kaffpa() call usesint*
The text was updated successfully, but these errors were encountered: