[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Getfem-commits] please merge branch devel-tetsuo-add-format-to-vecs

From: Yves Renard
Subject: Re: [Getfem-commits] please merge branch devel-tetsuo-add-format-to-vecsave
Date: Fri, 16 Nov 2018 13:49:22 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1


Yes, I didn't detect that. If boost/format was a self-sufficient file, we could integrate it, but it depends of many other header boost files. So it would be preferable to have another solution, yes.

Best regards,

Le 16/11/2018 à 01:10, Tetsuo Koyama a écrit :

Thank you very much for your feedback. 

Maybe using printf is better. Because It can set format by character.

If I could get agreement, I will make branch again.

Best Regards Tetsuo

I see this has now been merged to master. It seems though that it adds a new dependency on boost/format. As far as I remember we do not have a hard dependency on boost. If this is the case, I don't think it is a good idea to add such a dependency. Isn't there any alternative for adding such a format argument to vecsave without the boost library?

Best regards

On Tue, Nov 13, 2018 at 2:28 PM Tetsuo Koyama <address@hidden> wrote:
Dear GetFEM++ Project

I added format option to vecsave in GMM.
I inspired in numpy's savetxt.

You can set any format you want by format

Could you merge it to master branch or comment somthing?
I think this option is useful, but I also welcome dissenting opinions.

I checked it in my local coding source and also did a regression test
in repository.

Best regard Tetsuo


  Yves Renard (address@hidden)       tel : (33)
  20, rue Albert Einstein
  69621 Villeurbanne Cedex, FRANCE


reply via email to

[Prev in Thread] Current Thread [Next in Thread]