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

sequentialDVI: check-problem. Add seed, mutation #25

Closed
thoree opened this issue Jan 15, 2021 · 6 comments
Closed

sequentialDVI: check-problem. Add seed, mutation #25

thoree opened this issue Jan 15, 2021 · 6 comments

Comments

@thoree
Copy link
Collaborator

thoree commented Jan 15, 2021

Fra/til Thore: Sjekk sequentialDVI. Legge til seed slik at resultater blir reproduserbare?
Hvordan håndteres mutasjoner?

library(dvir)
data(planecrash)
pm = planecrash$pm
am = planecrash$am
missing = planecrash$missing
sequentialDVI(pm, am, missing)
#> Error in checkDVI(pm, am, missing = missing, moves = moves): Third vector must a vector with names of missing persons

Created on 2021-01-15 by the reprex package (v0.3.0)

@magnusdv
Copy link
Owner

Generelt tror jeg checkDVI() fortjener en gjennomgang. Noen sjekker er unødvendige, mens andre mangler. Spesielt kan man vurdere om den skal kontrollere at markør-attributter er identiske mellom AM og PM.

@thoree
Copy link
Collaborator Author

thoree commented Jan 17, 2021

Enig, jeg skal se på checkDVI senere

@thoree
Copy link
Collaborator Author

thoree commented Jan 19, 2021

Problem med sequentialDVI ikke knyttet til checkDVI, feilmelding fra nMarkers():

library(dvir)
pm = example2$pm
am = example2$am
missing = example2$missing
sequentialDVI(pm, am, missing, check = F, verbose = T)
#> Iteration 1 
#>    M1 M2 M3
#> V1  1  0  0
#> V2  1  5  0
#> V3  0  0  5
#> --> V2 = M2
#> Iteration 2 
#>    M1 M3
#> V1 10  0
#> V3  0  5
#> --> V1 = M1
#> Iteration 3 
#>    M3
#> V3  5
#> --> V3 = M3
#> Error: Input to `nMarkers()` must be a `ped` object or a list of such

Created on 2021-01-19 by the reprex package (v0.3.0)

@magnusdv
Copy link
Owner

Ah, I see the problem. I suggest to insert the following line in singleSearch.R, just before ### Update the LR matrix (line 66):

if(i == length(RES)) 
  break 

@thoree
Copy link
Collaborator Author

thoree commented Jan 19, 2021

Thanks, problem solved! (I figured out that you meant in sequentialDVI.R)

@thoree thoree closed this as completed Jan 19, 2021
@magnusdv
Copy link
Owner

(Correct - sorry!)

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

No branches or pull requests

2 participants