microsoft / seismic-deeplearning

Deep Learning for Seismic Imaging and Interpretation
MIT License
403 stars 142 forks source link

debug test.py not limiting test output in debug mode - facilitates faster build runtimes #331

Open maxkazmsft opened 4 years ago

maxkazmsft commented 4 years ago

Too many images are being added to the test1 split dump in test.py – we should not be scoring the entire batch with debug flag with the code in test.py

    if debug:
        logger.info("Running in Debug/Test mode")
        test_loader = take(2, test_loader)

but somehow we are doing this anyway.

Make sure that we understand this behaviour, as with even 2 slices and slicing in the inline direction, we should not have so many patches which are dumped for scoring. This could be related to #330 as if we're scoring too many empty and largely padded patches, this functionality is not needed.

Restructuing: potentially remove "if debug" and subset the data for tests across the board.