Revision Quality
Revision quality describes how committed revisions in our Subversion Repository perform with the latest tests. Because the tests change, the measured quality is expected to change moving forward. Developers must write patches for problems that are discovered as a consequence to extending test coverage. These patches improve our revision quality. As we continue to improve our tests, the patches must be kept up-to-date so that there is a gradual and concurrent expansion of test coverage and functioning code. Note that patches are also stored in our Subversion Repository, so this document reflects the latest status according.
Ideally we'd like for all revisions to pass without incident (perfect quality), however this is a very unlikely scenario; we have better things to do than generate patch after patch to handle legacy code, such as extending functionality.
Our golden revision is r418, the revision prior to SFSU's involvement. The behavior of this revision is the standard by which we measure all other revisions. We may find bugs in r418, or simple improvements (such as data mining all PDBs for all 3-letter residue codes and addressing each of them). In these cases the choice for golden revision may be changed.
Revision |
Status |
Patch |
To Do |
418 |
Golden Revision |
|
|
419 - 453 |
Configure Script Fails |
|
Prepare a r419 patch |
454 - 473 |
OK |
|
|
474 - 481 |
OK |
r474 overloaded variable patch |
|
482 |
OK |
r482 uppercase parse functions patch |
|
483-484 |
OK |
r483 lowercase parse methods patch |
|
485 |
Build fails |
|
Nothing; bug fixed in r486 |
486-489 |
OK |
r483 lowercase parse functions patch |
|
490-492 |
OK |
r489 all lowercase parse functions patch |
|
493 |
Build fails |
|
Nothing; student learning to use SVN |
494-516 |
OK |
r494 all lowercase parse functions and string api patch |
|
517-524 |
OK |
r517 boundary checks patch |
|
525 |
Patch fails |
r494 |
|
540 |
OK |
|
|
541 |
Build fails |
|
Nothing; bug fixed in r542 |
542 |
buildmodel fails |
|
Nothing; patch not feasible (too many changes) |
543 |
OK |
|
(Reverted to last known good 540) |
544-637 |
OK |
|
|
638 |
Build test fails |
|
Nothing; forgot to update the Makefile, fixed in 639 |
639-current |
OK |
|
|
Note r521 fails with the new featurize.t exhaustive test (r520 works).