-
Notifications
You must be signed in to change notification settings - Fork 662
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
you may need a cast phase3.hpp #2
Comments
that's funny, I knew it was risky without a cast ;) |
I think it all depends on your expectations. |
can you debug it and send me stack trace or something? don't have a mac to test on. |
yep.. don't expect to get it working... `
|
Ah that's not a crash, it was throwing an exception, there should be some output in the terminal? |
basically nothing..... |
are you running out of disk space by any chance? or hitting a maximum allowed open file limit? |
Hi, so basically it warns at 256, but it's unlimited.... ... i found it , but I cannot figure out WHY!!!! I thought perhaps the file pointer was at end of file, so i added a : The only other thing i can think of is perhaps the file is not "flushed" closed & reopened.. Why would a file with 300 MB of data , that can be freely copied and reports the size as: but you can see i even grab the size of the file & save it is "size" |
The bucket files are closed in phase1.hpp:400, and re-opened for reading in DiskSort.hpp:183. Even if it wasn't closed beforehand it will be closed before opening again for reading in |
I found a bug that might be the cause of this, fix pending. |
Try latest master |
LOL.. extreme bug hunting.... latest master does not fix issue the error is still being thrown in "DiskSort" |
If size of transfer is an issue, try to lower the N: |
i'm beginning to think its related to "T::disk_size" of 9. the machine has 96GB ram.....so it's not a buffer allocation problem... |
Try setting it to 10 or 12 or 16, in phase1.h:36 |
ok its got it..... dropping it to 32768 fixed it.... [P1] Table 1 took 413.396 sec i'm also thinking that the drive for phase 1 , it writes 127 of 300MB, that is stuff that could be held in ram. I'm just wondering if it actually needs a memaloc for this buffer on osx, instead of a hard build. |
Ah maybe you got a stack-overflow from it... |
and onto the next one.... don't see how... there is 96gb of ram, i think maybe the implementation in the kernel for fread is bad |
ok same reason but in |
yeah that's a good idea, also I should allocate the buffers on heap, not on the stack. |
check |
MSVC bringup and other improvements
/Volumes/null/chia-plotter/include/chia/phase3.hpp:48:24: error: no matching function for call to 'min' const auto count = std::min(L_position - L_offset, L_buffer.size()); ^~~~~~~~ /usr/local/opt/llvm/bin/../include/c++/v1/algorithm:2594:1: note: candidate template ignored: deduced conflicting types for parameter '_Tp' ('unsigned long long' vs. 'unsigned long') min(const _Tp& __a, const _Tp& __b)
correction seems to be:
const auto count = std::min<unsigned long>(L_position - L_offset, L_buffer.size());
The text was updated successfully, but these errors were encountered: