jolibrain / joliGEN

Generative AI Image Toolset with GANs and Diffusion for Real-World Applications
https://www.joligen.com
Other
238 stars 31 forks source link

fix: empty test metrics and test batch size default to train batch size #593

Closed pierrot-lc closed 1 month ago

pierrot-lc commented 8 months ago

This PR fixes two issues :

  1. When we provide a test size that is lower than the train size, the model will try to pull some images that do not exists and the training will crash. See here.
  2. When we remove the test visualization, the code will still try to plot the test metrics, leading to another crash.

The first issue is still problematic, my fix will only remove the crash but it does not modify the current behaviour that will always try to visualize the same amount of images, whether we are visualizing the test or train images. It could be fix in another PR.