I think it will be best to talk through the code comments in person. Sneha and I can talk through this on Tuesday afternoon.

Notes from in-person review:
remove the code that was commented out in core/genometry/src/main/java/com/affymetrix/genometry/symloader/VCF.java
line 328

core/genometry/src/main/java/com/affymetrix/genometry/symloader/VCF.java
line 520

Otherwise, comments were addressed in discussion. No need for further testing. Remove the commented code and create pull request.

I made those changes, rebased onto upstream master, and created a pull request.

ann.loraine commented on pull request #623: IGBF-1380 Support VCF4.2

How is this more efficient? Does it reduce:

• processing time (less work per line of data)
• memory usage (how much?)
  It seems to me that the main value of this change is it reduces complexity of the code, making it easier to understand and maintain.

_________________

You are right. "more efficient" was not the right phrase. "simpler" is better.
I reduced the comments that I added for the INFO and FORMAT sub-class definitions in VCF.java.

Merged into master

My testing verifies that all VCF versions are now compatible with IGB and load as expected. There was no exception noted in the console during my testing and all other file formats are loading as expected, so this story will now be closed.