Conversation
images/makeproject/project/TODO
Outdated
There was a problem hiding this comment.
Can you maybe add something in this file describing that this specific task is for the diamond vein project?
I know that's what it is now since it's in the PR but if we continue to use a TODO like this, I'm hoping to avoid confusion next time I have to look at it
There was a problem hiding this comment.
Yeah I added the todo to keep track of remaining config tasks, now that project.xml is also updated I removed the TODO file altogether
...keproject/project/apps/diaveins/1.00/windows_x86_64__cuda__win-modern/diaveins-cuda_1.00.exe
Show resolved
Hide resolved
There was a problem hiding this comment.
Can you clarify how the credit number was determined?
There was a problem hiding this comment.
Also - I guess there are only 4096 unique tasks?
There was a problem hiding this comment.
Honestly I took the 50k credits I saw being used for loneliest seed and scaled it down arbitrarily to account for shorter tasks, I'll benchmark the tasks and estimate credits based on that instead.
There was a problem hiding this comment.
There should be 65k tasks though, each task does 4096 internal cuda jobs if i understand this correctly
There was a problem hiding this comment.
How did we determine the fpops est/bound?
There was a problem hiding this comment.
fpops/delay bound were just copied 1:1 from another project, estimate was based on a scuffed benchmark, I'll run the benchmark properly now so that the estimate is based on actual data
There was a problem hiding this comment.
Ok the fpops est & bound are scaled relative to repeatdecor values now: diaveins task takes around 2.08x as long as repeatdecor on my machine so I multiplied credits, fpops bound and fpops est by that as well
Source code for binaries:
https://github.com/MinecraftAtHome/diamond-ore-veins