-
Notifications
You must be signed in to change notification settings - Fork 10
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
Timings #3
Comments
Hi @wolffan, thanks for this report. 👍 So in your experience using the script result in longer build times? How much RAM memory do you have? Maybe the memory pressure introduced by the script (default is 4GB RAM drive) makes your Mac do more swapping. Or maybe you compiled first in memory, then in disc and magically something got cached. (Can be Core Data generated code already there?) Love to hear more details about the experiment: Xcode version, code you were compiling (Swift / Objective-C), if external dependencies were involved (CocoaPods) and if you have a link to a public to replicate it. Besides raw speed, other reasons for using this script are:
It would be a good idea to benchmark it using some test projects... |
The machine is MacBook 2015 with 16 GB of ram. Xcode 8.3.3 The experiments where done with the clean project and derived data folder deleted (I use Watchdog for this). So in theory both builds should be exactly equal. Yup using the script it takes longer to compile :( I did the experiment twice, and times vary some seconds but they are still on those ranges (for that project). If you're looking into projects maybe having the artsy could work? |
I will be testing it tomorrow with Xcode 9, so far I've noticed higher CPU usage. |
I've done some experiments.
Using the script and the ramdisk on the derived data folder compile time is over 8 minutes.
Using the normal config compile time is under 5 minutes.
This seems quite a high price to pay for using this.
I don't really know how to check the disk usage, the tools from Mac OS don't give much clue...
Thanks for the script :). I hope this bit of info is useful!
Raimon
The text was updated successfully, but these errors were encountered: