Task #628 (closed)
Reduce number of blitz c++ compilation units to speed up compile
| Reported by: | jamoore | Owned by: | jamoore |
|---|---|---|---|
| Priority: | minor | Milestone: | 3.0-Beta2 |
| Component: | Deployment | Version: | 3.0-Beta1 |
| Keywords: | blitz, codegen, cpp, iteration3 | Cc: | cxallan |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | n.a. |
Description
Currently, the same number of *.cpp and *.h files are generated as *.java files. These should be combined into a single model.cpp and model.h file. (This requires, however, some conditionals due to Permissions and Details. The codegen may need some reworking to make this viable.)
Change History (8)
comment:1 Changed 12 years ago by jmoore
- Cc callan added
comment:2 Changed 12 years ago by jmoore
Donde via r1383 but the build is still impressively slow. Other thoughts?
comment:3 Changed 12 years ago by jmoore
- Resolution set to fixed
- Status changed from new to closed
Donde? How about "done".
comment:4 Changed 12 years ago by jmoore
r1449 reverts this partially to make the build more distributable via distcc (ccache also working)
From 70+ minutes on my box, to 20+ minutes distributed, to 1-7 minutes cached!
comment:5 Changed 12 years ago by jmoore
- Resolution fixed deleted
- Status changed from closed to reopened
comment:6 Changed 12 years ago by jmoore
- Resolution set to wontfix
- Status changed from reopened to closed
r1498 rolls back this change. Didn't see any real speed up, plus it hinders using distcc (ice.cpp was a beast to compile), plus it makes working with the *.ice files (which should be our central API definition a la javadocs) nearly unusable. Instead, we'll ship a concat'ed header to simplify things.
comment:7 Changed 8 years ago by Melissa Linkert <melissa@…>
- Remaining Time set to 0
- Resolution changed from wontfix to fixed
(In [cbc27a96852aab3c953ed0848a0eb07d574eadbd/bioformats.git]) Added more flexible plate/well detection, so that InCell? 3000 datasets are picked up by ScreenReader?. Closes #628.
(cherry picked from commit c9b128f2649e593546f14995043e20a67c62a754)
comment:8 Changed 8 years ago by Melissa Linkert <melissa@…>
(In [cbc27a96852aab3c953ed0848a0eb07d574eadbd/bioformats.git]) Added more flexible plate/well detection, so that InCell? 3000 datasets are picked up by ScreenReader?. Closes #628.
(cherry picked from commit c9b128f2649e593546f14995043e20a67c62a754)
Just finished a first implementation of this. Took 40 minutes!?