X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fio%2Fvamsas%2FSequencemapping.java;h=a33390fc5a0ea29848141ddaa51a7f5e935eef2a;hb=57738a1f3c19b1c3a00bd3ac5108f8cd0af32f99;hp=160f5e27c5b3947fd58f75f7f64997a20a97a70d;hpb=a21b84a669a9620f6943b9b5054e8756cb6f0687;p=jalview.git diff --git a/src/jalview/io/vamsas/Sequencemapping.java b/src/jalview/io/vamsas/Sequencemapping.java index 160f5e2..a33390f 100644 --- a/src/jalview/io/vamsas/Sequencemapping.java +++ b/src/jalview/io/vamsas/Sequencemapping.java @@ -20,13 +20,17 @@ */ package jalview.io.vamsas; +import jalview.bin.Console; import jalview.datamodel.AlignedCodonFrame; import jalview.datamodel.AlignmentI; +import jalview.datamodel.DBRefEntry; import jalview.datamodel.Mapping; import jalview.datamodel.SequenceI; import jalview.gui.Desktop; import jalview.io.VamsasAppDatastore; +import java.util.ArrayList; +import java.util.List; import java.util.Vector; import uk.ac.vamsas.objects.core.AlignmentSequence; @@ -96,16 +100,14 @@ public class Sequencemapping extends Rangetype } if (from != null && sequenceMapping.getLoc() != from) { - jalview.bin.Cache.log.warn("Probable IMPLEMENTATION ERROR: " + from + Console.warn("Probable IMPLEMENTATION ERROR: " + from + " doesn't match the local mapping sequence."); } if (ds != null && sequenceMapping.is__stored_in_document() && sequenceMapping.getV_parent() != ds) { - jalview.bin.Cache.log - .warn("Probable IMPLEMENTATION ERROR: " - + ds - + " doesn't match the parent of the bound sequence mapping object."); + Console.warn("Probable IMPLEMENTATION ERROR: " + ds + + " doesn't match the parent of the bound sequence mapping object."); } } @@ -152,8 +154,8 @@ public class Sequencemapping extends Rangetype SequenceType to = (SequenceType) getjv2vObj(jvto); if (to == null) { - jalview.bin.Cache.log - .warn("FIXME NONFATAL - do a second update: Ignoring Forward Reference to seuqence not yet bound to vamsas seuqence object"); + Console.warn( + "FIXME NONFATAL - do a second update: Ignoring Forward Reference to seuqence not yet bound to vamsas seuqence object"); return; } SequenceMapping sequenceMapping = new SequenceMapping(); @@ -183,8 +185,8 @@ public class Sequencemapping extends Rangetype if (!dnaToProt) { - jalview.bin.Cache.log - .warn("Ignoring Mapping - don't support protein to protein mapping in vamsas document yet."); + Console.warn( + "Ignoring Mapping - don't support protein to protein mapping in vamsas document yet."); return; } if (ds == null) @@ -211,8 +213,8 @@ public class Sequencemapping extends Rangetype true); } ds.addSequenceMapping(sequenceMapping); - sequenceMapping.setProvenance(this - .dummyProvenance("user defined coding region translation")); // TODO: + sequenceMapping.setProvenance( + this.dummyProvenance("user defined coding region translation")); // TODO: // correctly // construct // provenance @@ -223,29 +225,26 @@ public class Sequencemapping extends Rangetype // mapping bindjvvobj(mjvmapping.getMap(), sequenceMapping); - jalview.bin.Cache.log.debug("Successfully created mapping " - + sequenceMapping.getVorbaId()); + Console.debug( + "Successfully created mapping " + sequenceMapping.getVorbaId()); } // private void update(jalview.util.MapList mjvmapping, // SequenceMapping sequenceMapping) { - jalview.bin.Cache.log - .error("Not implemented: Jalview Update Alcodon Mapping:TODO!"); + Console.error("Not implemented: Jalview Update Alcodon Mapping:TODO!"); } private void update(SequenceMapping sequenceMapping, jalview.datamodel.Mapping mjvmapping) { - jalview.bin.Cache.log - .error("Not implemented: Update DBRef Mapping from Jalview"); + Console.error("Not implemented: Update DBRef Mapping from Jalview"); } private void update(jalview.datamodel.Mapping mjvmapping, SequenceMapping sequenceMapping) { - jalview.bin.Cache.log - .error("Not implemented: Jalview Update Sequence DBRef Mapping"); + Console.error("Not implemented: Jalview Update Sequence DBRef Mapping"); } /** @@ -281,7 +280,7 @@ public class Sequencemapping extends Rangetype } if (sdloc == null || sdmap == null) { - jalview.bin.Cache.log.info("Ignoring non sequence-sequence mapping"); + Console.info("Ignoring non sequence-sequence mapping"); return; } mobj = this.getvObj2jv(sdloc); @@ -297,8 +296,8 @@ public class Sequencemapping extends Rangetype if (from == null || to == null) { - jalview.bin.Cache.log - .error("Probable Vamsas implementation error : unbound dataset sequences involved in a mapping are being parsed!"); + Console.error( + "Probable Vamsas implementation error : unbound dataset sequences involved in a mapping are being parsed!"); return; } @@ -388,33 +387,40 @@ public class Sequencemapping extends Rangetype { if (from.getDBRefs() == null && to.getDBRefs() == null) { - if (jalview.bin.Cache.log.isDebugEnabled()) + if (Console.isDebugEnabled()) { - jalview.bin.Cache.log.debug("Not matching conjugate refs for " - + from.getName() + " and " + to.getName()); + Console.debug("Not matching conjugate refs for " + from.getName() + + " and " + to.getName()); } return; } - if (jalview.bin.Cache.log.isDebugEnabled()) + if (Console.isDebugEnabled()) { - jalview.bin.Cache.log.debug("Matching conjugate refs for " - + from.getName() + " and " + to.getName()); + Console.debug("Matching conjugate refs for " + from.getName() + + " and " + to.getName()); } - jalview.datamodel.DBRefEntry[] fdb = from.getDBRefs(); - jalview.datamodel.DBRefEntry[] tdb = new jalview.datamodel.DBRefEntry[to - .getDBRefs().length]; - int tdblen = to.getDBRefs().length; - System.arraycopy(to.getDBRefs(), 0, tdb, 0, tdblen); + List fdb = from.getDBRefs(); + List tdb = new ArrayList(to.getDBRefs()); + int tdblen = to.getDBRefs().size(); + // + // + // YOWSER + // + // System.arraycopy(to.getDBRefs(), 0, tdb, 0, tdblen); + // + // + // + // Vector matched = new Vector(); jalview.util.MapList smapI = smap.getInverse(); - for (int f = 0; f < fdb.length; f++) + for (int f = 0, fn = fdb.size(); f < fn; f++) { - jalview.datamodel.DBRefEntry fe = fdb[f]; + jalview.datamodel.DBRefEntry fe = fdb.get(f); jalview.datamodel.Mapping fmp = fe.getMap(); - boolean fmpnnl = fmp != null; + boolean fmpnnl = (fmp != null); // if (fmpnnl && fmp.getTo()!=null) // { - // jalview.bin.Cache.log.debug("Not overwriting existing To reference in + // Cache.debug("Not overwriting existing To reference in // "+fe); // continue; // } @@ -426,7 +432,7 @@ public class Sequencemapping extends Rangetype : false; for (int t = 0; t < tdblen; t++) { - jalview.datamodel.DBRefEntry te = tdb[t]; + jalview.datamodel.DBRefEntry te = tdb.get(t); if (te != null) { if (fe.getSource().equals(te.getSource()) @@ -442,8 +448,9 @@ public class Sequencemapping extends Rangetype boolean smaptolocal2tm = (tmpnnl) ? smap.equals(tmp.getMap()) : false; // smap to maps from te.map to te.local - boolean smaptotemap2local = (tmpnnl) ? smapI.equals(fmp - .getMap()) : false; + boolean smaptotemap2local = (tmpnnl) + ? smapI.equals(fmp.getMap()) + : false; if (smapfromlocal2fe && smaptotemap2local) { // smap implies mapping from to to from