Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Can't initialise OCPD with more than 1 dimension #1

Open
multimeric opened this issue Feb 19, 2021 · 1 comment · May be fixed by #4
Open

Can't initialise OCPD with more than 1 dimension #1

multimeric opened this issue Feb 19, 2021 · 1 comment · May be fixed by #4

Comments

@multimeric
Copy link

If you simply have dims > 1, and no other parameters, you get:

require('ocp')
#> Loading required package: ocp
empty_ocpd <- initOCPD(dims=5)
#> Error in initProb[[dimi]]: subscript out of bounds

Created on 2021-02-19 by the reprex package (v1.0.0)

It's not possible to manually provide the initProb input, either, because gaussian_init is not exported:

require('ocp')
#> Loading required package: ocp
empty_ocpd <- initOCPD(dims=5, initProb=rep(ocp::gaussian_init, 5))
#> Error: 'gaussian_init' is not an exported object from 'namespace:ocp'

Created on 2021-02-19 by the reprex package (v1.0.0)
@multimeric multimeric mentioned this issue Feb 19, 2021
Open
@anjapago
Copy link
Owner

anjapago commented Mar 4, 2021

thanks, if you have a fix to improve feel free to open a PR

@multimeric multimeric linked a pull request Mar 4, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix initialization and incompatible settings bugs multimeric/ocp-1
2 participants
@multimeric @anjapago