You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was trying to record msprime.NodeType.MIGRANT in the SweepGenicModel. In theory, this should allow us to record lineages that recombine away from the selected background (= changing labels in the structured coalescent).
This returns the following error: msprime._msprime.LibraryError: The simulation model supplied resulted in a parent node having a time value <= to its child. This can occur as a result of multiple bottlenecks happening at the same time, multiple census events at the same time or numerical imprecision with very smallpopulation sizes.
This is due to how the sweep is set up: msp_sweep_initialise. The move individuals function is applied here to the sampled lineages. We shouldn't be registering an additional node here. The same error will happen when the sweep is finalised I think.
The text was updated successfully, but these errors were encountered:
In the long run, recording events this way may cause some issues. Arguably, one goal is for msprime to replace, e.g., discoal, in terms of functionality. If that happens, then the notion of a "migration" will get overloaded: between fitness classes and between "real" demes.
Yes, agreed. We need a new msprime.NodeType, or a way of recording fitness classes in a flexible way. In the mean time I will make sure we throw an error when trying to record MIGRANTS for the SweepGenicModel.
I was trying to record
msprime.NodeType.MIGRANT
in theSweepGenicModel
. In theory, this should allow us to record lineages that recombine away from the selected background (= changing labels in the structured coalescent).This returns the following error:
msprime._msprime.LibraryError: The simulation model supplied resulted in a parent node having a time value <= to its child. This can occur as a result of multiple bottlenecks happening at the same time, multiple census events at the same time or numerical imprecision with very smallpopulation sizes.
This is due to how the sweep is set up:
msp_sweep_initialise
. The move individuals function is applied here to the sampled lineages. We shouldn't be registering an additional node here. The same error will happen when the sweep is finalised I think.The text was updated successfully, but these errors were encountered: