-
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
Many tests fail on 10.6.8 Rosetta [ FAIL 32 | WARN 0 | SKIP 0 | PASS 78 ] #57
Comments
Yeah, I think know what this is. Part of the craziness with supporting annotations across subplots is factoring in padding/margins so that the right coordinates can be inferred. IIRC, R will bug out if certain fonts do not fit within those margins and I don't know if I can pre-specify a specific font. Can you share some details? Was this a straight up stock run? |
@miyamot0 I ran a version from CRAN, but I have just tried from the latest commit, and results are identical. (No patches were applied to the source, if that is what you mean.) (These were in Rosetta; let me try on a native PowerPC machine, but normally they are identical in terms of failures.) |
@miyamot0 Ha, no errors on PowerPC system! Rosetta errors are not worth wasting time on, to be honest. But I wonder what goes wrong on it. (PPC machine has a slightly newer GCC, 12.3.0, but this unlikely to matter here.) |
UPD. Hmm, why total number of tests differs? 81 pass here (0 everything else), 78 pass + 32 failures on Rosetta. |
Hmm, I think I may have written tests poorly (still new to unit testing in R). I'll dig into this if there may be a concurrency issue. In my experience, variability between tests is usually my fault. Thanks for the follow-up -- much appreciated! |
@miyamot0 Thank you very much! |
UPD. Native PowerPC build and tests appear good (see below).
P. S.
figure margins too large
is a known error (though its cause is unknown to me), so not specific to the package.The text was updated successfully, but these errors were encountered: