commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yannick TANGUY (JIRA)" <>
Subject [jira] [Commented] (MATH-811) Improve event detection by selecting g function slope
Date Wed, 03 Oct 2012 07:18:08 GMT


Yannick TANGUY commented on MATH-811:

Your idea seems interesting, but I think it depends on how users already handle event detection.
If most of them won't use the slope selection, maybe an extended interface is better, or an
abstract class (like in OREKIT events package), so that these users don't have to implement
this method.
But if there are "enough" users who can be interested in the new features, I think it must
be implemented in EventHandler interface, and users will have to adapt their code.
In the second hypothesis, I understand it will have to wait for 4.0.

Anyway, what's really important for us is to have this feature now (and it's ok, "our" version
of Commons Math implement it), to identify precisely the differences between our version and
the official version (snapshot or a released version) and then, to know approximatively when
this feature could be integrated (Commons Math roadmap).

By now, this discussion has enabled to improve the implementation, clarify the use of event
detection : that's a good point ! And when you'll have time to look further into it, we can
discuss again to find a more elegant solution (if any) and integrate it.

> Improve event detection by selecting g function slope
> -----------------------------------------------------
>                 Key: MATH-811
>                 URL:
>             Project: Commons Math
>          Issue Type: Improvement
>    Affects Versions: 3.1
>            Reporter: Yannick TANGUY
>         Attachments: JIRA-811_slope_selection.patch, JIRA-811_slope_selection_V2.patch,
JIRA-811_slope_selection_V3.patch, JIRA-811_slope_selection_V3_with_spaces.patch, JIRA-811_slope_selection_with_spaces.patch
> We would like to select g function slope in CommonsMath event detection feature.
> This would improve event detection for the final user (the possibility to ignore some
events, and less computation time) and also correct a bug found while using OREKIT (see
in our applications. 
> The modification consists in adding a method (getSlopeSelection() : int) in EventHandler,
and modifying one conditionnal in the method evaluateStep (class EventState). 
> We can provide modified code and unitary tests for this feature.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message