MIScnn icon indicating copy to clipboard operation
MIScnn copied to clipboard

steps per epoch update when self.prepare_batches

Open MLRadfys opened this issue 4 years ago • 1 comments

Hi again,

one more thing I noticed (sorryyyy):

When you use patchwise_grid and set the prepare batches flag, I think that the number of steps per epoch is not updated correctly.

You set the length to self.batchpointer, but didn't account for the batch size:

steps_per_epoch = math.ceil(len(self.batchpointers) / self.batch_size)

Maybe I missed something though,

Yes indeed, I did...

The batch size is already included in your when you increase the batchpointer in the run method....

Regards,

Michael

MLRadfys avatar Jun 08 '20 11:06 MLRadfys

Hey MichaelLempart,

one more thing I noticed (sorryyyy):

No need to apologize! I'm very happy for any feedback to improve the utilization of MIScnn and especially thankful that someone is thoroughly testing the package :)

I will have a look on it.

Cheers, Dominik

muellerdo avatar Jun 09 '20 08:06 muellerdo