Line 1: |
Line 1: |
− | === Review Sept 17th ===
| + | == Review Sept 17th == |
| + | === Topics Discussed === |
| + | * [[#Return Statuses|Checking if methods succeeded/failed (checking return values/return statuses)]] |
| + | * [[#Accessing String Values|Strings as return values]] |
| + | |
| + | === NOTES From Meeting === |
| *InputFile should not use <code>long int</code>. Should instead use: <code>long long</code> | | *InputFile should not use <code>long int</code>. Should instead use: <code>long long</code> |
| *Anytime have an error could call handleError which would have a switch to return the error, throw exception, or abort. Call it with an error code and a string. Maybe an error handler class where you could use everywhere. Each class would have a member of that class type that would contain that information. | | *Anytime have an error could call handleError which would have a switch to return the error, throw exception, or abort. Call it with an error code and a string. Maybe an error handler class where you could use everywhere. Each class would have a member of that class type that would contain that information. |
| | | |
− | ==== Review Discussion Topics ==== | + | ==== Useful Links ==== |
− | http://genome.sph.umich.edu/wiki/SAM/BAM_Library_FAQs | + | BAM Library FAQs: http://genome.sph.umich.edu/wiki/SAM/BAM_Library_FAQs |
| + | |
| + | Source Code: http://www.sph.umich.edu/csg/mktrost/doxygen/html/ |
| | | |
− | http://www.sph.umich.edu/csg/mktrost/doxygen/html/ | + | Test code for setting values in the library: http://www.sph.umich.edu/csg/mktrost/doxygen/html/WriteFiles_8cpp-source.html |
| | | |
− | Example of using the library to set values: http://www.sph.umich.edu/csg/mktrost/doxygen/html/WriteFiles_8cpp-source.html
| + | === Topics for Discussion === |
− | ===== Return Statuses =====
| + | ==== Return Statuses ==== |
| Currently anytime you do anything on a SAM/BAM file, you have to check the status for failure: | | Currently anytime you do anything on a SAM/BAM file, you have to check the status for failure: |
| <source lang="cpp"> | | <source lang="cpp"> |
Line 67: |
Line 74: |
| | | |
| | | |
− | ===== Accessing String Values =====
| + | ==== Accessing String Values ==== |
| SAM/BAM files have strings in them that people will want to read out. | | SAM/BAM files have strings in them that people will want to read out. |
| How should we handle this interface? | | How should we handle this interface? |
Line 137: |
Line 144: |
| http://www.sph.umich.edu/csg/mktrost/doxygen/html/SamRecord_8h-source.html | | http://www.sph.umich.edu/csg/mktrost/doxygen/html/SamRecord_8h-source.html |
| | | |
− | ===== SamFileHeader =====
| + | ==== SamFileHeader ==== |
| *Should this be renamed to SamHeader? | | *Should this be renamed to SamHeader? |
| *Do we like the classes being named starting with Sam? Should it be Bam? | | *Do we like the classes being named starting with Sam? Should it be Bam? |
Line 177: |
Line 184: |
| Should these also be added to SamHeaderRG, SamHeaderSQ, etc as appropriate.... | | Should these also be added to SamHeaderRG, SamHeaderSQ, etc as appropriate.... |
| | | |
− | === Review June 7th ===
| + | == Review June 7th == |
| | | |
| * <S>Move the examples from the SamFile wiki page to their own page</s> | | * <S>Move the examples from the SamFile wiki page to their own page</s> |