deepdive icon indicating copy to clipboard operation
deepdive copied to clipboard

"deepdive run" does less than "deepdive do all"

Open alldefector opened this issue 8 years ago • 3 comments

In my workflow, I often want/need to examine the weight table (dd_inference_result_weights_mapping), but "deepdive run" doesn't generate that. Instead, I have to run "deepdive do all" after "deepdive run" every time...

alldefector avatar Jul 16 '16 22:07 alldefector

deepdive redo init/app weights will do the job.

deepdive run was mainly kept for backward compatibility, but I guess we can refresh it to be something like: deepdive mark todo init/app && deepdive do all

netj avatar Jul 16 '16 23:07 netj

@netj can you please confirm that init/app will not drop the database in any case?

I see deepdive db init in init/app/run.sh. I don't think this command is supposed to drop the database because db-init in both postgres and greenplum drivers doesn't dropdb unless the number of argument is greater than zero. But I did experience some cases when it did drop the db. Sorry can't recall the exact setting but was using the master branch a couple days ago.

xiaoling avatar Jul 18 '16 19:07 xiaoling

It should.. I believe the behavior dropping db hasn't been fixed yet.

netj avatar Jul 18 '16 19:07 netj