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.

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

Latin Hypercube sampling design (LHS) #33

Merged
merged 4 commits into from
Jul 25, 2021
Merged

Conversation

joergbuchwald
Copy link
Contributor

add a Latin Hypercube sampling design constructed by LatinHypercubeSampling.jl

@phrb phrb merged commit c07e299 into phrb:master Jul 25, 2021
@joergbuchwald
Copy link
Contributor Author

do you have an idea how to fix the pipeline? The output is nondeterministic. Does it make sense in your opinion to remove the no-deterministic part in the show overload?

@phrb
Copy link
Owner

phrb commented Jul 26, 2021

I use a seed at the documentation makefile in order to get deterministic results.
Seeded tests are standard practice, and we can use a set of seeds to try more variation.

You can build docs locally to get the proper output by doing:

$ cd ExperimentalDesign
$ julia

Then:

(@v1.6) pkg> activate docs
  Activating environment at `~/.julia/dev/ExperimentalDesign/docs/Project.toml`

julia> using Documenter, ExperimentalDesign

julia> random_seed = 443591

julia> DocMeta.setdocmeta!(ExperimentalDesign,
                           :DocTestSetup,
                           :(using ExperimentalDesign, Distributions,
                             Random, StatsModels, DataFrames;
                             Random.seed!($random_seed););
                           recursive = true)

These are the steps on the docs makefile, and will allow you to test the current version:

julia> doctest(ExperimentalDesign)

To update the doctests with this seed, run:

julia> doctest(ExperimentalDesign; fix = true)

This will change your local files, so make sure all other tests pass and commit the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants