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
byres and same residue as selects the whole residue for each atom in the subsequent selection, by the canonical and unique resindex. Likewise same segment as uses segindex.
Actual behavior
byres and same residue as use the user-supplied, potentially repeated resid values. same segment as uses segid. This is redundant as you can already use same resid as or same segid as for the same behaviour. This may result in unexpected behaviour with altLocs also.
Code to reproduce the behavior
Leads to problems like in this mailing list issue where selecting "byres resname TIP3" returns various amino acids and ions as well.
* fixes#2669
* fixes#2672
* fixes "same residue" and "same segment" selections
* Use resindices of resids for same residue selection
* Use segindices of segids for same segment selection
* Use resindices for byres selection
* add failing test
* failing tests with correct number of atoms
* fix same selection
* add failing byres test
* fix byres test
* add comments
* updated changelog
Expected behavior
byres
andsame residue as
selects the whole residue for each atom in the subsequent selection, by the canonical and unique resindex. Likewisesame segment as
usessegindex
.Actual behavior
byres
andsame residue as
use the user-supplied, potentially repeatedresid
values.same segment as
usessegid
. This is redundant as you can already usesame resid as
orsame segid as
for the same behaviour. This may result in unexpected behaviour with altLocs also.Code to reproduce the behavior
Leads to problems like in this mailing list issue where selecting "byres resname TIP3" returns various amino acids and ions as well.
Current version of MDAnalysis
python -c "import MDAnalysis as mda; print(mda.__version__)"
) 0.20.1python -V
)? 3.7.3The text was updated successfully, but these errors were encountered: