Merge pull request #377 from Serrof/issue-376

fixed visibility issue in interpolators for non-stiff integrators

Author: Serrof

hipparchus-ode/src/main/java/org/hipparchus/ode/AbstractFieldIntegrator.java

@@ -43,6 +43,7 @@
 import org.hipparchus.ode.events.FieldODEStepEndHandler;
 import org.hipparchus.ode.events.FieldStepEndEventState;
 import org.hipparchus.ode.sampling.AbstractFieldODEStateInterpolator;
+import org.hipparchus.ode.sampling.FieldODEStateInterpolator;
 import org.hipparchus.ode.sampling.FieldODEStepHandler;
 import org.hipparchus.util.FastMath;
 import org.hipparchus.util.Incrementor;
@@ -221,13 +222,13 @@ protected FieldODEStateAndDerivative<T> initIntegration(final FieldExpandableODE
                         eqn.getMapper().mapStateAndDerivative(t0, y0, y0Dot);
 
         // initialize detector based event states (both  and step end based)
-        detectorBasedEventsStates.stream().forEach(s -> {
+        detectorBasedEventsStates.forEach(s -> {
             s.init(s0WithDerivatives, t);
             s.getEventDetector().getHandler().init(s0WithDerivatives, t, s.getEventDetector());
         });
 
         // initialize step end based event states
-        stepEndEventsStates.stream().forEach(s -> {
+        stepEndEventsStates.forEach(s -> {
             s.init(s0WithDerivatives, t);
             s.getHandler().init(s0WithDerivatives, t);
         });
@@ -310,12 +311,12 @@ protected FieldODEStateAndDerivative<T> acceptStep(final AbstractFieldODEStateIn
 
         FieldODEStateAndDerivative<T> previousState = interpolator.getGlobalPreviousState();
         final FieldODEStateAndDerivative<T> currentState = interpolator.getGlobalCurrentState();
-        AbstractFieldODEStateInterpolator<T> restricted = interpolator;
+        FieldODEStateInterpolator<T> restricted = interpolator;
 
         // initialize the events states if needed
         if (!statesInitialized) {
             // initialize event states
-            detectorBasedEventsStates.stream().forEach(s -> s.reinitializeBegin(interpolator));
+            detectorBasedEventsStates.forEach(s -> s.reinitializeBegin(interpolator));
             statesInitialized = true;
         }
 
@@ -339,7 +340,7 @@ public int compare(FieldEventState<T> es0, FieldEventState<T> es1) {
 
             // Evaluate all event detectors and end steps for events
             occurringEvents.clear();
-            final AbstractFieldODEStateInterpolator<T> finalRestricted = restricted;
+            final FieldODEStateInterpolator<T> finalRestricted = restricted;
             Stream.concat(detectorBasedEventsStates.stream(), stepEndEventsStates.stream()).
             forEach(s -> { if (s.evaluateStep(finalRestricted)) {
                     // the event occurs during the current step

hipparchus-ode/src/main/java/org/hipparchus/ode/AbstractIntegrator.java

@@ -41,6 +41,7 @@
 import org.hipparchus.ode.events.ODEStepEndHandler;
 import org.hipparchus.ode.events.StepEndEventState;
 import org.hipparchus.ode.sampling.AbstractODEStateInterpolator;
+import org.hipparchus.ode.sampling.ODEStateInterpolator;
 import org.hipparchus.ode.sampling.ODEStepHandler;
 import org.hipparchus.util.FastMath;
 import org.hipparchus.util.Incrementor;
@@ -81,7 +82,7 @@ public abstract class AbstractIntegrator implements ODEIntegrator {
     private Incrementor evaluations;
 
     /** Differential equations to integrate. */
-    private transient ExpandableODE equations;
+    private ExpandableODE equations;
 
     /** Build an instance.
      * @param name name of the method
@@ -208,13 +209,13 @@ protected ODEStateAndDerivative initIntegration(final ExpandableODE eqn,
                         eqn.getMapper().mapStateAndDerivative(t0, y0, y0Dot);
 
         // initialize detector based event states (both  and step end based)
-        detectorBasedEventsStates.stream().forEach(s -> {
+        detectorBasedEventsStates.forEach(s -> {
             s.init(s0WithDerivatives, t);
             s.getEventDetector().getHandler().init(s0WithDerivatives, t, s.getEventDetector());
         });
 
         // initialize step end based event states
-        stepEndEventsStates.stream().forEach(s -> {
+        stepEndEventsStates.forEach(s -> {
             s.init(s0WithDerivatives, t);
             s.getHandler().init(s0WithDerivatives, t);
         });
@@ -292,18 +293,18 @@ protected ODEStateAndDerivative acceptStep(final AbstractODEStateInterpolator in
 
         ODEStateAndDerivative previousState = interpolator.getGlobalPreviousState();
         final ODEStateAndDerivative currentState = interpolator.getGlobalCurrentState();
-        AbstractODEStateInterpolator restricted = interpolator;
+        ODEStateInterpolator restricted = interpolator;
 
 
         // initialize the events states if needed
         if (!statesInitialized) {
             // initialize event states
-            detectorBasedEventsStates.stream().forEach(s -> s.reinitializeBegin(interpolator));
+            detectorBasedEventsStates.forEach(s -> s.reinitializeBegin(interpolator));
             statesInitialized = true;
         }
 
         // set end of step
-        stepEndEventsStates.stream().forEach(s -> s.setStepEnd(currentState.getTime()));
+        stepEndEventsStates.forEach(s -> s.setStepEnd(currentState.getTime()));
 
         // search for next events that may occur during the step
         final int orderingSign = interpolator.isForward() ? +1 : -1;
@@ -322,7 +323,7 @@ public int compare(final EventState es0, final EventState es1) {
 
             // Evaluate all event detectors and end steps for events
             occurringEvents.clear();
-            final AbstractODEStateInterpolator finalRestricted = restricted;
+            final ODEStateInterpolator finalRestricted = restricted;
             Stream.concat(detectorBasedEventsStates.stream(), stepEndEventsStates.stream()).
             forEach(s -> { if (s.evaluateStep(finalRestricted)) {
                     // the event occurs during the current step

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsBashforthFieldIntegrator.java

@@ -29,6 +29,7 @@
 import org.hipparchus.linear.FieldMatrix;
 import org.hipparchus.ode.FieldEquationsMapper;
 import org.hipparchus.ode.FieldODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsFieldStateInterpolator;
 import org.hipparchus.util.FastMath;
 
 

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsBashforthIntegrator.java

@@ -22,6 +22,7 @@
 import org.hipparchus.linear.RealMatrix;
 import org.hipparchus.ode.EquationsMapper;
 import org.hipparchus.ode.ODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsStateInterpolator;
 import org.hipparchus.util.FastMath;
 
 

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsFieldIntegrator.java

@@ -34,6 +34,7 @@
 import org.hipparchus.ode.FieldODEStateAndDerivative;
 import org.hipparchus.ode.LocalizedODEFormats;
 import org.hipparchus.ode.MultistepFieldIntegrator;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsFieldStateInterpolator;
 import org.hipparchus.util.MathArrays;
 
 

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsIntegrator.java

@@ -27,6 +27,7 @@
 import org.hipparchus.ode.MultistepIntegrator;
 import org.hipparchus.ode.ODEState;
 import org.hipparchus.ode.ODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsStateInterpolator;
 
 
 /** Base class for {@link AdamsBashforthIntegrator Adams-Bashforth} and

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsMoultonFieldIntegrator.java

@@ -34,6 +34,7 @@
 import org.hipparchus.ode.FieldEquationsMapper;
 import org.hipparchus.ode.FieldODEStateAndDerivative;
 import org.hipparchus.ode.LocalizedODEFormats;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsFieldStateInterpolator;
 import org.hipparchus.util.MathArrays;
 import org.hipparchus.util.MathUtils;
 

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/AdamsMoultonIntegrator.java

@@ -27,6 +27,7 @@
 import org.hipparchus.ode.EquationsMapper;
 import org.hipparchus.ode.LocalizedODEFormats;
 import org.hipparchus.ode.ODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.AdamsStateInterpolator;
 import org.hipparchus.util.FastMath;
 
 

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/ClassicalRungeKuttaFieldIntegrator.java

@@ -26,6 +26,7 @@
 import org.hipparchus.Field;
 import org.hipparchus.ode.FieldEquationsMapper;
 import org.hipparchus.ode.FieldODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.ClassicalRungeKuttaFieldStateInterpolator;
 import org.hipparchus.util.MathArrays;
 
 /**
@@ -111,7 +112,7 @@ public T[] getB() {
                            final FieldODEStateAndDerivative<T> globalPreviousState,
                            final FieldODEStateAndDerivative<T> globalCurrentState,
                            final FieldEquationsMapper<T> mapper) {
-        return new ClassicalRungeKuttaFieldStateInterpolator<T>(getField(), forward, yDotK,
+        return new ClassicalRungeKuttaFieldStateInterpolator<>(getField(), forward, yDotK,
                                                                globalPreviousState, globalCurrentState,
                                                                globalPreviousState, globalCurrentState,
                                                                mapper);

hipparchus-ode/src/main/java/org/hipparchus/ode/nonstiff/ClassicalRungeKuttaIntegrator.java

@@ -19,6 +19,7 @@
 
 import org.hipparchus.ode.EquationsMapper;
 import org.hipparchus.ode.ODEStateAndDerivative;
+import org.hipparchus.ode.nonstiff.interpolators.ClassicalRungeKuttaStateInterpolator;
 
 /**
  * This class implements the classical fourth order Runge-Kutta
@@ -85,15 +86,12 @@ public double[] getB() {
 
     /** {@inheritDoc} */
     @Override
-    protected ClassicalRungeKuttaStateInterpolator
-    createInterpolator(final boolean forward, double[][] yDotK,
-                       final ODEStateAndDerivative globalPreviousState,
-                       final ODEStateAndDerivative globalCurrentState,
-                       final EquationsMapper mapper) {
-        return new ClassicalRungeKuttaStateInterpolator(forward, yDotK,
-                                                       globalPreviousState, globalCurrentState,
-                                                       globalPreviousState, globalCurrentState,
-                                                       mapper);
+    protected ClassicalRungeKuttaStateInterpolator createInterpolator(final boolean forward, final double[][] yDotK,
+                                                                      final ODEStateAndDerivative globalPreviousState,
+                                                                      final ODEStateAndDerivative globalCurrentState,
+                                                                      final EquationsMapper mapper) {
+        return new ClassicalRungeKuttaStateInterpolator(forward, yDotK, globalPreviousState, globalCurrentState,
+                                                       globalPreviousState, globalCurrentState, mapper);
     }
 
 }