Replace this method with SeqFeatureI.isSameFeat which
does the same thing + tests the feat ID
isSameFeat was needed to correct a bug with split
transcripts.
yea - a refactoring - either scrap RangeI.isIdentical or
rename it isSameRange (for consistency/clarity) and have
isSameFeat call it and add the id check - i kinda like the
latter idea - im gonna lower the priority on this one - not
crucial but would be nice - also there is a place or 2 where
is isIdentical is called
also its unclear what "group" this gets - im unclear how to
use group - oh also category - i think for category this
should either be a new category - "datamodel" or editing
should be renamed editing/datamodel - still feeling this out
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
yea - a refactoring - either scrap RangeI.isIdentical or
rename it isSameRange (for consistency/clarity) and have
isSameFeat call it and add the id check - i kinda like the
latter idea - im gonna lower the priority on this one - not
crucial but would be nice - also there is a place or 2 where
is isIdentical is called
also its unclear what "group" this gets - im unclear how to
use group - oh also category - i think for category this
should either be a new category - "datamodel" or editing
should be renamed editing/datamodel - still feeling this out
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=422804
yea - a refactoring - either scrap RangeI.isIdentical or
rename it isSameRange (for consistency/clarity) and have
isSameFeat call it and add the id check - i kinda like the
latter idea - im gonna lower the priority on this one - not
crucial but would be nice - also there is a place or 2 where
is isIdentical is called
also its unclear what "group" this gets - im unclear how to
use group - oh also category - i think for category this
should either be a new category - "datamodel" or editing
should be renamed editing/datamodel - still feeling this out
Logged In: YES
user_id=422804
yea - a refactoring - either scrap RangeI.isIdentical or
rename it isSameRange (for consistency/clarity) and have
isSameFeat call it and add the id check - i kinda like the
latter idea - im gonna lower the priority on this one - not
crucial but would be nice - also there is a place or 2 where
is isIdentical is called
also its unclear what "group" this gets - im unclear how to
use group - oh also category - i think for category this
should either be a new category - "datamodel" or editing
should be renamed editing/datamodel - still feeling this out