-
Notifications
You must be signed in to change notification settings - Fork 0
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
print gang information #10
Labels
Comments
garfieldnate
added a commit
that referenced
this issue
Mar 21, 2021
Specifically, trying to save the supracontext label to BasicSupra. Used JOL to check if we had room in BasicSupra for the extra label field, and it seems to be totally fine: # Objects are 8 bytes aligned. # Field sizes by type: 4, 1, 1, 2, 2, 4, 4, 8, 8 [bytes] # Array element sizes: 4, 1, 1, 2, 2, 4, 4, 8, 8 [bytes] weka.classifiers.lazy.AM.data.BasicSupra object internals: OFFSET SIZE TYPE DESCRIPTION VALUE 0 12 (object header) N/A 12 4 java.math.BigInteger BasicSupra.count N/A 16 4 java.util.Set BasicSupra.data N/A 20 4 (loss due to the next object alignment) Instance size: 24 bytes Space losses: 0 bytes internal + 4 bytes external = 4 bytes total However, getting the correct label in the distributed lattice, in a performant way, will take some thinking... Will fix #10.
garfieldnate
added a commit
that referenced
this issue
Mar 22, 2021
Specifically, trying to save the supracontext label to BasicSupra. Used JOL to check if we had room in BasicSupra for the extra label field, and it seems to be totally fine: # Objects are 8 bytes aligned. # Field sizes by type: 4, 1, 1, 2, 2, 4, 4, 8, 8 [bytes] # Array element sizes: 4, 1, 1, 2, 2, 4, 4, 8, 8 [bytes] weka.classifiers.lazy.AM.data.BasicSupra object internals: OFFSET SIZE TYPE DESCRIPTION VALUE 0 12 (object header) N/A 12 4 java.math.BigInteger BasicSupra.count N/A 16 4 java.util.Set BasicSupra.data N/A 20 4 (loss due to the next object alignment) Instance size: 24 bytes Space losses: 0 bytes internal + 4 bytes external = 4 bytes total However, getting the correct label in the distributed lattice, in a performant way, will take some thinking... Will fix #10.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Went looking for the answer to #9 because Dave Eddington wanted to see proper gangs output. Discovered the below:
D'oh!
The text was updated successfully, but these errors were encountered: