Closed fissell closed 4 years ago
Hi Kate,
I would recommend you sync DSI Studio and TIPL at the same time. The
version from different time frame may not pass the compiler.
The recent change I did to DSI Studio is only the syntax part. It does not involve any computation update. Whether you use the last week rev and current version should be consistent.
Best regards, Frank
On Tue, Nov 19, 2013 at 11:00 AM, fissell notifications@github.com wrote:
Hi Frank,
The Nov 18 rev of TIPL is not compiling on Linux as part of DSI_Studio build. Is it safe to take last week's DSI_Studio rev and last week's TIPL rev ? How closely do I need to sync DSI_Studio and TIPL revs ?
thanks Kate
../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:233:10: error: ‘pixel_index’ was not declared in this scope ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:233:44: error: ‘iter’ was not declared in this scope ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:606:13: error: ‘pixel_index’ was not declared in this scope ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:606:40: error: ‘index’ does not have class type ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:606:69: error: ‘index’ does not have class type ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:607:25: error: ‘index’ does not have class type ../DSI-Studio_18Nov2013_Src/image/numerical/basic_op.hpp:607:43: error: ‘index’ does not have class type ../DSI-Studio_18Nov2013_Src/image/numerical/optimization.hpp:25:43: error: ‘class image::pixel_index<2u>’ has no member named ‘valid’ ../DSI-Studio_18Nov2013_Src/image/numerical/dif.hpp:131:45: error: ‘class image::pixel_index<3u>’ has no member named ‘valid’ ../DSI-Studio_18Nov2013_Src/image/numerical/dif.hpp:192:45: error: ‘class image::pixel_index<3u>’ has no member named ‘valid’ ../DSI-Studio_18Nov2013_Src/image/numerical/dif.hpp:251:45: error: ‘class image::pixel_index<2u>’ has no member named ‘valid’ ../DSI-Studio_18Nov2013_Src/image/numerical/dif.hpp:283:45: error: ‘class image::pixel_index<2u>’ has no member named ‘valid’
— Reply to this email directly or view it on GitHubhttps://github.com/frankyeh/TIPL/issues/1 .
Thanks Frank ! Is the TIPL compile problem at my end or at your end ? I cloned DSI-Studio and TIPL this morning.
Kate
I will do the commit again to make sure that the codes are updated.
Best, Frank
On Tue, Nov 19, 2013 at 11:12 AM, fissell notifications@github.com wrote:
Thanks Frank ! Is the TIPL compile problem at my end or at your end ? I cloned DSI-Studio and TIPL this morning.
Kate
— Reply to this email directly or view it on GitHubhttps://github.com/frankyeh/TIPL/issues/1#issuecomment-28803075 .
Hi,
I think I got your latest updates (Nov 19) but I am getting the same compile errors:
basic_op.hpp:233:10: error: pixel_index was not declared in this scope
and few more in basic_op.hpp
and I think optimization.hpp and dif.hpp didn't get the pixel updates:
optimization.hpp:25:43: error: class image::pixel_index has no member named valid
thanks Kate
Hi Kate,
Thank for reporting these errors. I corrected them and uploaded the
update. It passed my compiler, and please compile it again to see if there is any error at your side.
Best regards, Frank
On Tue, Nov 19, 2013 at 1:12 PM, fissell notifications@github.com wrote:
Hi,
I think I got your latest updates (Nov 19) but I am getting the same compile errors:
basic_op.hpp:233:10: error: pixel_index was not declared in this scope
and few more in basic_op.hpp and I think optimization.hpp and dif.hpp didn't get the pixel updates: optimization.hpp:25:43: error: €˜class image::pixel_index€™ has no member named €˜valid™
thanks Kate
— Reply to this email directly or view it on GitHubhttps://github.com/frankyeh/TIPL/issues/1#issuecomment-28816652 .
Thanks very much Frank !!! It compiles now, we're testing !
Hi Frank,
The Nov 18 rev of TIPL is not compiling on Linux as part of DSI_Studio build. Is it safe to take last week's DSI_Studio rev and last week's TIPL rev ? How closely do I need to sync DSI_Studio and TIPL revs ?
thanks Kate