[
https://issues.apache.org/jira/browse/MATH-1597?focusedWorklogId=720728&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-720728
]
ASF GitHub Bot logged work on MATH-1597:
----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Feb/22 08:57
Start Date: 04/Feb/22 08:57
Worklog Time Spent: 10m
Work Description: samyBadjoudj closed pull request #190:
URL: https://github.com/apache/commons-math/pull/190
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 720728)
Time Spent: 2h 50m (was: 2h 40m)
> Refactor low-discrepancy sequence functionality
> -----------------------------------------------
>
> Key: MATH-1597
> URL: https://issues.apache.org/jira/browse/MATH-1597
> Project: Commons Math
> Issue Type: Improvement
> Reporter: Gilles Sadowski
> Priority: Minor
> Fix For: 4.X
>
> Time Spent: 2h 50m
> Remaining Estimate: 0h
>
> Two low-discrepancy sequences are implemented in Commons Math (in package
> {{o.a.c.math4.legacy.random}}:
> * {{SobolSequenceGenerator}}
> * {{HaltonSequenceGenerator}}
> They both provide several methods:
> * {{nextVector()}}
> * {{skipTo(int)}}
> * {{getNextIndex()}}
> of which only the first is part of their common API through the
> {{RandomVectorGenerator}}.
> I propose to create an interface that would better represent the specific
> concept (and avoid the confusion with pseudo-random generators):
> {code}
> public interface LowDiscrepancySequence extends Supplier<double[]> { /* ...
> */}
> {code}
> Thus, instead of class {{SobolSequenceGenerator}} we'd have:
> {code}
> public class SobolSequence implements LowDiscrepancySequence {
> // ...
> }
> {code}
> This functionality could be moved to a new {{o.a.c.m.legacy.quasirandom}}
> package (?).
> Method {{skipTo}} could be replaced with an API similar to Commons RNG
> {{JumpableUniformRandomProvider}} (?).
> The new design (TBD on the "dev" ML) should ideally take [additional
> implementations|https://en.wikipedia.org/wiki/Low-discrepancy_sequence] into
> account.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
