Skip to content

Commit

Permalink
Merge pull request #2115 from vgteam/glenn2
Browse files Browse the repository at this point in the history
fix validate -x and mod --retain-complement
  • Loading branch information
glennhickey authored Feb 18, 2019
2 parents b0b8815 + ba39fe8 commit ec6929e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/subcommand/mod_main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,7 @@ int main_mod(int argc, char** argv) {
{"orient-forward", no_argument, 0, 'O'},
{"unfold", required_argument, 0, 'f'},
{"retain-path", required_argument, 0, 'r'},
{"retain-complement", no_argument, 0, 'I'},
{"subgraph", required_argument, 0, 'g'},
{"context", required_argument, 0, 'x'},
{"remove-null", no_argument, 0, 'R'},
Expand Down
7 changes: 4 additions & 3 deletions src/subcommand/validate_main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@

#include "../vg.hpp"
#include "../alignment.hpp"
#include "../stream/vpkg.hpp"

using namespace std;
using namespace vg;
Expand Down Expand Up @@ -120,12 +121,12 @@ int main_validate(int argc, char** argv) {
cerr << "error:[vg validate] -n, -e -o, -p cannot be used with -a and -x" << endl;
return 1;
}
xg::XG xindex;
ifstream in(xg_path.c_str());
xindex.load(in);
unique_ptr<xg::XG> xindex = stream::VPKG::load_one<xg::XG>(in);
in.close();
get_input_file(gam_path, [&](istream& in) {
stream::for_each<Alignment>(in, [&](Alignment& aln) {
if (!alignment_is_valid(aln, &xindex)) {
if (!alignment_is_valid(aln, xindex.get())) {
exit(1);
}
});
Expand Down

2 comments on commit ec6929e

@adamnovak
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for merge to master. View the full report here.

19 tests passed, 0 tests failed and 0 tests skipped in 17302 seconds

Tests produced 2486 warnings. 2486 were for lower-than-expected alignment scores

@adamnovak
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for branch v1.14.0. View the full report here.

19 tests passed, 0 tests failed and 0 tests skipped in 17598 seconds

Tests produced 2488 warnings. 2488 were for lower-than-expected alignment scores

Please sign in to comment.