search

gaynorr / AlphaSimR

R package for breeding program simulations
https://gaynorr.github.io/AlphaSimR/
Other
42 stars 18 forks source link

Fixing #83 by adding another nInd=1 corner case in genParam() #86

Closed gregorgorjanc closed 1 year ago

gregorgorjanc commented 1 year ago

The issue #83 revealed another nInd=1 corner case in genParam()

gaynorr commented 1 year ago

I decided to address this in popVar instead, just incase this edge case pops up somewhere else.

gregorgorjanc commented 1 year ago

Yeah, fixing popVar would be indeed better.

Note that we added a patch to genParam for such a case already (so make sure you remove that part then too).

With regards, Gregor

On Wed, Sep 28, 2022 at 00:44:49, Chris Gaynor @.***> wrote:

I decided to address this in popVar instead, just incase this edge case pops up somewhere else.

— Reply to this email directly, view it on GitHub https://github.com/gaynorr/AlphaSimR/pull/86#issuecomment-1260150003, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFRGSR6VIRBRH3BDRDSXALWAN2GDANCNFSM6AAAAAAQWC7HNA . You are receiving this because you authored the thread.Message ID: @.***>