ncbi-vdb icon indicating copy to clipboard operation
ncbi-vdb copied to clipboard

Recognize pre-release tables

Open durbrow opened this issue 1 year ago • 1 comments

This code is not wrong. As far as KDB is concerned, it is correct. It is the same logic that KDBPathType uses, and KDBPathContents should match KDBPathType. But as far as I can tell, we have no files that are so old that KDBPathType would recognize them as pre-release tables.

durbrow avatar Oct 04 '24 20:10 durbrow

The problem with the ones from the SRAO ticket is that KDB does not recognize them as pre-release tables. I thought they'd work for a test case but I was wrong. That's why I don't have a test case. :(

You can pass it or fail it. It is not testable (and neither is KDBPathType in this case). Maybe it's time to deprecate the concept of a pre-release KDB table.

durbrow avatar Oct 05 '24 15:10 durbrow