Re: [boost] [Review] ODEInt review starts today, September 19, ends September 28

Subject: Re: [boost] [Review] ODEInt review starts today, September 19, ends September 28
From: Karsten Ahnert (karsten.ahnert_at_[hidden])
Date: 2012-09-21 04:17:19

Hi Denis,

thank you very much for your review.

On 09/20/2012 01:42 PM, Denis Demidov wrote:

>> - What is your evaluation of the implementation?
> I only had a quick glance at the implementation, but the code looks
> decent, clean
> and well formatted.  Implementation of some algebras and operations
> could probably
> benefit from usage of C++11 variadic templates, but this would have to
> wait for wider
> adoption of the new standard.
We have plans to provide a C++11 version of odeint facing this point
(and several other). It would also allow to express many operations on
the state types in terms of transform instead of for_each.
>> - What is your evaluation of the documentation?
> Documentation is good. Examples in tutorial cover most use cases and common
> state types and operations.  There is an overview of avalable steppers, but
> more detailed description of each stepper (typical use cases, stability
> requirements, advantages and disadvantages w.r.t. other methods) would probably
> be beneficial.
Ok. This is a good point. I think we can easily provide a more detailed
description of the steppers.