Re: [ITK-dev] [ITK] Patches for the 4.8.2 release

2015-11-09 Thread Dženan Zukić
Better submit the usual way. I doubt Matt will include it without the usual testing. Add me and Matt as reviewers. Regards, Dženan On Mon, Nov 9, 2015 at 3:18 PM, Simon Alexander wrote: > We found this today (patch attached). If 4.8.2 is right around the > corner, can you consider it for incl

Re: [ITK-dev] BUILD_EXAMPLES default to OFF

2015-11-16 Thread Dženan Zukić
+1 On Fri, Nov 13, 2015 at 7:02 PM, Johnson, Hans J wrote: > +1 too. > > > > > > On 11/13/15, 5:46 PM, "Insight-developers on behalf of Matt McCormick" < > insight-developers-boun...@itk.org on behalf of matt.mccorm...@kitware.com> > wrote: > > >Hi folks, > > > >In order to improve the first imp

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-11-25 Thread Dženan Zukić
t; * And much more! See details in the log below. >> >> >> ^ >> >> >> >> >> >> >> >> >> >> >> >> Changes from v4.8.0..v4.9rc01 >> >> >> --

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-11-25 Thread Dženan Zukić
until Saturday, but have hopes for a few free hours there. > > On Wed, 25 Nov 2015 at 15:58 Dženan Zukić wrote: > >> Building git master from Nov 23rd with examples and testing enabled I get >> 14 compile errors in VS2015 debug x64. Should we coordinate our efforts in >>

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-11-25 Thread Dženan Zukić
t; > On Wed, 25 Nov 2015 at 16:06 Dženan Zukić wrote: > >> OK I will work on it for an hour or two now, and I will make a branch and >> push it to Gerrit so whoever wants can pick up from there and keep updating >> it. >> >> On Wed, Nov 25, 2015 at 4:02 PM, Simon

Re: [ITK-dev] Gerrit login

2015-11-27 Thread Dženan Zukić
Hi Paolo, clicking on your link works for me. Have you tried clearing cookies for that domain? You might be logged in with the wrong account. Regards, Dženan On Fri, Nov 27, 2015 at 5:21 AM, Paolo Zaffino wrote: > Dear all, > I am facing some problems with gerrit login. > I approved the github

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-11-30 Thread Dženan Zukić
ere useful I can help with it now? Not sure what state we > are in (or what branch). > > On Wed, 25 Nov 2015 at 16:41 Dženan Zukić wrote: > >> It turns out that all the compile errors were occurring in WikiExamples. >> I solved all except one: >> ITK_FIXED_PARAMETER

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-12-01 Thread Dženan Zukić
; http://review.source.kitware.com/#/c/20446/ >> >> >> And Dzenan and Bill have addressed Wiki examples errors: >> >> https://github.com/InsightSoftwareConsortium/ITKWikiExamples/pull/3 >> >> >> It is likely that the test failures on your build still remain

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.9 Release Candidate 1 is ready for testing!

2015-12-08 Thread Dženan Zukić
Thanks Simon for getting to this. All changes look simple enough. You maybe needed to amend previous commits on the same topic to generate just one review. Regards, Dženan On Tue, Dec 8, 2015 at 4:31 PM, Simon Alexander wrote: > I've sent a patch set to gerrit, managed to screw it up somehow a

Re: [ITK-dev] itkStaticAssert warnings with clang

2016-01-04 Thread Dženan Zukić
Hi Victor, this is an excellent opportunity for you to contribute to ITK the patch which fixes the described issue. Add Matt McCormick as one of the reviewers. Regards, Dženan On Mon, Jan 4, 2016 at 8:30 AM, Poughon Victor wrote: > Hi all, > > I think t

Re: [ITK-dev] itkStaticAssert warnings with clang

2016-01-05 Thread Dženan Zukić
wiki. > Maybe it's related to the issue my colleague was having: > http://permalink.gmane.org/gmane.comp.lib.itk.devel/3159 > > So here is a format-patch. > > Thanks, > > Victor Poughon > -- > *De :* Dženan Zukić [dzen...@gmail.com]

Re: [ITK-dev] Possible to build ITK libs without version number in filename?

2016-02-06 Thread Dženan Zukić
Hi Sean, version numbers shouldn't be an issue if you configure your project using CMake (which automatically takes care of all library paths). I forgot how much of a pain in the ass it is to set those paths without CMake - something one project reminded me of this week. HTH, Dženan On Fri, Feb

[ITK-dev] Bug in Maurer filter

2016-03-25 Thread Dženan Zukić
Hi everyone, I found a bug in Maurer distance map filter . Can anyone familiar with this filter take a look? Regards, Dženan _

Re: [ITK-dev] [ITK] Community versus Users Lists

2016-04-02 Thread Dženan Zukić
12:16 PM, Matt McCormick < > matt.mccorm...@kitware.com> wrote: > >> Doodle polls are usually for scheduling times only (as far as I am >> aware). Perhaps a Google Form? >> >> M >> >> On Tue, Mar 29, 2016 at 11:10 AM, Dženan Zukić wrote: >>

Re: [ITK-dev] problems with programs crashing -- potentially due to multiple versions of ITK?

2016-04-29 Thread Dženan Zukić
Hi Tim, the classic solution for this is to build ITK (and VTK) as static libraries. I have a dozen of different ITK versions on my computer and they coexist nicely. Regards On Fri, Apr 29, 2016 at 3:51 PM, Matt McCormick wrote: > > Thanks for the quick reply. I assumed that's what I was doin

Re: [ITK-dev] installed ITK cannot be moved due to absolute path

2016-05-10 Thread Dženan Zukić
Hi Rashad, a few months ago relative paths were completely removed from CMake, because they were never fully implemented. Have you followed recommendations from here for cross-compiling? As for integrating your changes into ITK, you should submit a patch

Re: [ITK-dev] ITK4.10rc1 SimpleITK VNL error

2016-05-18 Thread Dženan Zukić
Hi Hans, Python support is included with default installation of VS2015. You just need to tick it when installing, or modifying your installation. Or were you asking about something different? Regards On Tue, May 17, 2016 at 11:07 AM, Johnson, Hans J wrote: > Thanks Brad. Do you know if this

[ITK-dev] c:\.git\hooks

2016-07-22 Thread Dženan Zukić
Hi everybody, occasionally, a folder "C:\.git\hooks" appears on my computer. Since I use TortoiseGit, this is very annoying because right-click context menu of entire C: drive thinks it is under version control. I filed a bug report, but sup

Re: [ITK-dev] c:\.git\hooks

2016-07-25 Thread Dženan Zukić
09:00 AM, Dženan Zukić wrote: > > occasionally, a folder "C:\.git\hooks" appears on my computer. > [snip] > > but support people think that git hooks in some of my projects > > (almost all instances of ITK, VTK and Slicer builds) are causing > > this behavior. Did an

[ITK-dev] ITK wrapping with VS2013

2016-08-01 Thread Dženan Zukić
Hi Brad and everybody, I am trying to build ITK with Ultrasound module, FFTW and wrapping for Python. This builds without errors on Linux, but on Windows I run into issues: 165> Building Custom Rule C:/Libs/InsightToolkit-4.10.0/Modules/External/ITKUltrasound/wrapping/CMakeLists.txt ... 165>C:\L

Re: [ITK-dev] Unable to login to review.source.kitware.com

2016-08-16 Thread Dženan Zukić
Hi Taylor, did the issue go away by now? Regards, Dženan On Mon, Aug 15, 2016 at 6:32 PM, Taylor Braun-Jones wrote: > On Mon, Aug 15, 2016 at 6:21 PM, Taylor Braun-Jones > wrote: > > > > I'm trying to login to Kitware's Gerrit instance in order to submit an > ITK patch and getting the followi

Re: [ITK-dev] ITKv5

2016-09-09 Thread Dženan Zukić
I added a couple items to the wishlist. On Fri, Sep 9, 2016 at 3:28 PM, Francois Budin wrote: > Hello everyone, > > I just started a wishlist page for ITKv5. As far as I know, there is no > current ITKv5 officially planned but it may be a good location to keep > track of the current limitations

Re: [ITK-dev] Pre-commit Failure uncrustify

2016-09-12 Thread Dženan Zukić
sh >> >> On Mon, Sep 12, 2016 at 9:11 PM, Dzenan Zukic >> wrote: >> >>> Hi Somesh, >>> >>> You need a recent master version (2016-08-26 or newer) which you can >>> build yourself from source available on GitHub >>> <https://gi

Re: [ITK-dev] Pre-commit Failure uncrustify

2016-09-14 Thread Dženan Zukić
must conclude that you are using somebody's fork of uncrustify, not HEAD from the official repository. Can you verify this? Regards, Dženan On Mon, Sep 12, 2016 at 10:17 PM, Dženan Zukić wrote: > It is not required. > > Dženan@phone > > On Sep 12, 2016 22:15, "Somesh

Re: [ITK-dev] Pre-commit Failure uncrustify

2016-09-14 Thread Dženan Zukić
5145d5bd4108f1043 Merge pull request #661 >from uncrustify/NewLineProt > > > I can take a look into why its not working for me in the weekend. > > Thanks, > Somesh > > > > On Wed, Sep 14, 2016 at 4:04 PM, Dženan Zukić wrote: > >> Hi Somes

[ITK-dev] lhcp-vm-rh502.nlm

2016-09-19 Thread Dženan Zukić
Hi guys, who owns build machine lhcp-vm-rh502.nlm? It looks like this machine needs to have it build directory cleaned up, it produces some spurious warnings such as these . Regards, Dženan _

Re: [ITK-dev] itk::LinearInterpolateImageFunction::EvaluateAtContinuousIndex() -- what is intended behaviour outside of image?

2016-09-20 Thread Dženan Zukić
Hi Steve, this function is not supposed to do bounds checking. Bounds checking can be done with IsInsideBuffer(). The commit which introduced the code: Revision: 6abbc7969a90786c4c73f5d191f634db536c2d1d Author: Hans Johnson Date: 2010-08-05 22:35:42 Message: Made USE_OPTIMIZED_REGISTRATION the d

Re: [ITK-dev] [ITK] description AdaptiveHistogramEqualizationImageFilter

2016-10-25 Thread Dženan Zukić
Dear Johannes, this is an excellent opportunity to contribute to ITK by improving the documentation! Following these instructions , modify the comments

Re: [ITK-dev] Visual Studio 2017 RC

2016-12-06 Thread Dženan Zukić
I have. It has internal compiler error a few hundred times when compiling ITK. I submitted the issue to Microsoft. Beta 3 or 4 (I don't remember which one I tried) compiled ITK without issues - but that version was still using the old compiler toolchain (v140). 2017 RC uses toolset v141. Regards,

[ITK-dev] Fwd: [Developer Community] Ulzii Luvsanbat posted a new comment on "Compiling ITK triggers internal error"

2016-12-07 Thread Dženan Zukić
-- Forwarded message -- From: Date: Wed, Dec 7, 2016 at 6:17 PM Subject: [Developer Community] Ulzii Luvsanbat posted a new comment on "Compiling ITK triggers internal error" To: dzen...@gmail.com <http://www.visualstudio.com> Hi dženan-zukić, A new comment on

Re: [ITK-dev] [ITK] [ITK-users] Fwd: [Developer Community] Ulzii Luvsanbat posted a new comment on "Compiling ITK triggers internal error"

2016-12-08 Thread Dženan Zukić
de generated has any differences or problems. I wonder > if there is a little hack that could be done to work around this ICE issue > in itkPrometeType.h, to see if there are any other issues later one… > > Brad > > On Dec 7, 2016, at 6:37 PM, Dženan Zukić wrote: > > > --

[ITK-dev] Fwd: [Developer Community] Ulzii Luvsanbat posted a new solution to "Compiling ITK triggers internal error"

2016-12-08 Thread Dženan Zukić
I guess they are nearing release. -- Forwarded message -- From: Date: Dec 8, 2016 13:23 Subject: [Developer Community] Ulzii Luvsanbat posted a new solution to "Compiling ITK triggers internal error" To: Cc: <http://www.visualstudio.com> > > Hi dženan-zuk

Re: [ITK-dev] installed ITK cannot be moved due to absolute path

2016-12-20 Thread Dženan Zukić
ossible for the module system to learn relative >> locations of these libraries with ITK_USE_SYSTEM*. But, it would work >> best if these projects were configured with CMake, create a >> Config.cmake or -config.cmake file, and use relative >> paths internally. >&

[ITK-dev] Too fast?

2017-01-04 Thread Dženan Zukić
Does somebody know why is this test now failing? https://open.cdash.org/testDetails.php?test=513022422&build=4708890 ___ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Kitware off

Re: [ITK-dev] Remote module AnisotropicDiffusionLBR compile error

2017-01-04 Thread Dženan Zukić
On it. On Wed, Jan 4, 2017 at 1:03 PM, Bill Lorensen wrote: > Now that the remote module has been fixed, someone needs to update > AnisotropicDiffusionLBR.remote.cmake to reference the repaired commit. > > > On Mon, Jan 2, 2017 at 12:41 PM, Bill Lorensen > wrote: > > This commit causes a compil

Re: [ITK-dev] Remote module AnisotropicDiffusionLBR compile error

2017-01-04 Thread Dženan Zukić
​Oh, that was already done by Francois 2 days ago.​ On Wed, Jan 4, 2017 at 2:03 PM, Dženan Zukić wrote: > On it. > > On Wed, Jan 4, 2017 at 1:03 PM, Bill Lorensen > wrote: > >> Now that the remote module has been fixed, someone needs to update >> AnisotropicD

[ITK-dev] vcpkg

2017-01-09 Thread Dženan Zukić
Hi guys, vcpkg looks like a good effort for easing library installation for usage in Visual Studio. Should we join the effort by adding ITK and VTK? Has anyone tried/started d

Re: [ITK-dev] [vtkusers] vcpkg

2017-01-10 Thread Dženan Zukić
o port ITK (and VTK?) to that platform. Since it supports CMake [1], it may not be much work. Francois [1] https://github.com/Microsoft/vcpkg/blob/master/docs/ EXAMPLES.md#example-1-2-b On Tue, Jan 10, 2017 at 10:15 AM, Ben Boeckel wrote: > On Mon, Jan 09, 2017 at 18:58:40 -0500, Dženan Zukić

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.11 Release Candidate 2 is ready for testing!

2017-01-19 Thread Dženan Zukić
ibraries >> BUG: Ensure CreateImageIO is concurrent thread safe >> ENH: Mark HDF5 CMake options as internal >> COMP: Add template export declaration for ParametricPath >> COMP: Correct TEMPLATE_EXPLICIT definition in Path module >> ENH: Factor

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.11 Release Candidate 2 is ready for testing!

2017-01-19 Thread Dženan Zukić
017 at 12:46 PM, Dženan Zukić wrote: > Hi Taylor, > > I submitted a bug > <https://developercommunity.visualstudio.com/content/problem/4679/compiling-itk-triggers-internal-error.html> > report to Microsoft about it, and somebody said they will add ITK to their > compiler

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.11 Release Candidate 2 is ready for testing!

2017-01-19 Thread Dženan Zukić
nks, Ulzii Luvsanbat Visual C++ Team On Thu, Jan 19, 2017 at 1:21 PM, Dženan Zukić wrote: > A new reply in that bug report: > > Hi, there's a lag when we, the compiler team, tags the bug resolved and > the fix actually gets included in the VS release. We apologize for th

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.11 Release Candidate 2 is ready for testing!

2017-01-29 Thread Dženan Zukić
RC3 got released on Friday, and it builds ITK without issue! On Thu, Jan 19, 2017 at 5:00 PM, Taylor Braun-Jones wrote: > Cool, thanks Dženan! > > On Thu, Jan 19, 2017 at 2:14 PM, Dženan Zukić wrote: > >> ... and more: >> >> Additionally, we've added ITK a

Re: [ITK-dev] JIRA down

2017-02-06 Thread Dženan Zukić
The "disk is full" was probably the root cause. Over the weekend somebody also reported CAPTCHA problem which was due to full disk too. On Mon, Feb 6, 2017 at 12:10 PM, Taylor Braun-Jones wrote: > It's back up for me and my co-workers as well. If it was something that > just fixed itself it migh

Re: [ITK-dev] [ITK-users] [ANN] ITK 4.11 Release Candidate 2 is ready for testing!

2017-03-10 Thread Dženan Zukić
Braun-Jones wrote: > Confirmed on my end as well. Thanks for following up. > > On Sun, Jan 29, 2017 at 4:41 PM, Dženan Zukić wrote: > >> RC3 got released on Friday, and it builds ITK without issue! >> >> On Thu, Jan 19, 2017 at 5:00 PM, Taylor Braun-Jones < >&g

Re: [ITK-dev] Build errors using a different C and CXX compiler

2017-04-07 Thread Dženan Zukić
Microsoft already did the same! https://developercommunity.visualstudio.com/comments/6266/view.html ITK is an excellent stress test case for compilers because of big code size and omnipresent templates :) Regards, Dženan On Fri, Apr 7, 2017 at 1:49 PM, Lefman, Jonathan wrote: > There is a conf

Re: [ITK-dev] Update on cmake support for TBB

2017-04-13 Thread Dženan Zukić
Hi Jonathan, that is great to hear! Is there any news on adding ITK to Intel compiler's regression test suite? Regards, Dženan Zukić, PhD, Senior R&D Engineer, Kitware (Carrboro, N.C.) On Thu, Apr 13, 2017 at 3:00 PM, Lefman, Jonathan wrote: > Hi all, > > I recently caug

Re: [ITK-dev] itk::Statistics::ScalarImageToCooccurrenceMatrixFilter input image and mask are same type

2017-06-07 Thread Dženan Zukić
//itk.org/Wiki/ITK/Git/Develop> a patch so others can benefit from this improvement. Regards, Dženan Zukić, PhD, Senior R&D Engineer, Kitware (Carrboro, N.C.) On Wed, Jun 7, 2017 at 9:48 AM, Yi Gao wrote: > Dear All, > > I was using the itk::Statistics::ScalarImageToCooccurrenc

Re: [ITK-dev] Patches for 4.12.2 release

2017-09-29 Thread Dženan Zukić
Hi Sean, I guess your use case is close to one-of-a-kind. And if we expose just one function, that makes little to no sense. Unless others have a different opinion, I think it is best if you copy-pasted it into your app. Regards, Dženan On Fri, Sep 29, 2017 at 11:39 AM, Sean McBride wrote: > O

Re: [ITK-dev] Problem calculating oriented bounding box, itk::ShapeLabelObject

2017-11-21 Thread Dženan Zukić
Hi Elli, can you repost on discourse ? We have moved there. Regards, Dženan On Tue, Nov 21, 2017 at 10:12 AM, Pfaehler, EAG (ngmb) < e.a.g.pfaeh...@umcg.nl> wrote: > Dear all, > > I need the size of the oriented bounding box of a mask. > I created a shape label objec

Re: [ITK-dev] Patches for the ITK 4.13 Release Candidate 1

2017-12-04 Thread Dženan Zukić
Hi Niels, the RC1 tag will be added to the current master, which means all the patches which have been merged to master will be included. Regards, Dženan On Mon, Dec 4, 2017 at 5:08 AM, Niels Dekker wrote: > Francois Budin wrote on December 1st 2017: > >> We are planning to tag ITK 4.13 Releas

Re: [ITK-dev] Patches for the ITK 4.13 Release Candidate 1

2017-12-04 Thread Dženan Zukić
din wrote: > >> Thanks Niels for the feedback. When we tag the RC1, please let us know >> if we missed any of these patch. >> >> On Mon, Dec 4, 2017 at 10:20 AM, Dženan Zukić >> wrote: >> >> Hi Niels, >>> >>> the RC1 tag will be added