ddp: Export sa_indices #255

Merged
merged 1 commit into from May 18, 2016

Projects

None yet

3 participants

@oyamad
Member
oyamad commented May 3, 2016

No description provided.

@coveralls

Coverage Status

Coverage decreased (-0.09%) to 86.604% when pulling 7459915 on sa_indices into fb42379 on master.

@oyamad oyamad added a commit to QuantEcon/QuantEcon.notebooks that referenced this pull request May 3, 2016
@oyamad oyamad Change _sa_indices to sa_indices 93ace31
@oyamad oyamad added a commit to QuantEcon/QuantEcon.notebooks that referenced this pull request May 3, 2016
@oyamad oyamad Change _sa_indices to sa_indices f450611
@mmcky
Contributor
mmcky commented May 17, 2016

Hi @oyamad. This looks like a nice refactor of the code into a utilities subpackage.
Are you happy for this to be merged alongside QuantEcon/QuantEcon.notebooks#37 and QuantEcon/QuantEcon.notebooks#36

@oyamad
Member
oyamad commented May 17, 2016

@mmcky Yes, please.

@mmcky mmcky merged commit e73730c into master May 18, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mmcky mmcky deleted the sa_indices branch May 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment