From 27eaa322fbf0c86cb2fa26f3ea38940a96a5feff Mon Sep 17 00:00:00 2001
From: Grant Sanderson <grant@3blue1brown.com>
Date: Tue, 6 Mar 2018 13:39:59 -0800
Subject: [PATCH] Change redered filed directory names

Default behavior is now to include the render quality in the name of the directory where files are written, so that low quality tests don't necessarily clobber the existing HD versions.
---
 scene/scene.py | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/scene/scene.py b/scene/scene.py
index 5168db25..04d2ddf3 100644
--- a/scene/scene.py
+++ b/scene/scene.py
@@ -40,7 +40,7 @@ class Scene(Container):
         "random_seed" : 0,
         "start_at_animation_number" : None,
         "end_at_animation_number" : None,
-        "include_render_quality_in_name" : False, #TODO, nothing uses this right now
+        "include_render_quality_in_output_directory" : True,
     }
     def __init__(self, **kwargs):
         Container.__init__(self, **kwargs) # Perhaps allow passing in a non-empty *mobjects parameter?
@@ -56,8 +56,6 @@ class Scene(Container):
         self.original_skipping_status = self.skip_animations
         if self.name is None:
             self.name = self.__class__.__name__
-            if self.include_render_quality_in_name:
-                self.name += str(self.camera.pixel_shape[0])
         if self.random_seed is not None:
             random.seed(self.random_seed)
             np.random.seed(self.random_seed)
@@ -552,15 +550,22 @@ class Scene(Container):
         image.save(path)
 
     def get_movie_file_path(self, name = None, extension = None):
+        directory = self.output_directory
+        if self.include_render_quality_in_output_directory:
+            directory += "_%dp%d"%(
+                self.camera.pixel_shape[0],
+                int(1.0/self.frame_duration)
+            )
+        if not os.path.exists(directory):
+            os.makedirs(directory)
+
         if extension is None:
             extension = self.movie_file_extension
         if name is None:
             name = self.name
-        file_path = os.path.join(self.output_directory, name)
+        file_path = os.path.join(directory, name)
         if not file_path.endswith(extension):
             file_path += extension
-        if not os.path.exists(self.output_directory):
-            os.makedirs(self.output_directory)
         return file_path
 
     def open_movie_pipe(self):