From a85c0cc1d498234a17941f8c483c69983b367529 Mon Sep 17 00:00:00 2001
From: "Olivier J.N. Bertrand" <olivier.bertrand@uni-bielefeld.de>
Date: Mon, 11 Dec 2017 19:01:50 +0100
Subject: [PATCH] update todo list
---
todo | 39 ++++++++++++++++++++++++++++++++++++++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/todo b/todo
index 5277215..e8885bc 100644
--- a/todo
+++ b/todo
@@ -1,3 +1,4 @@
+0001: Remove DUPLICATE:
Remove DUPLICATE: is_numeric_array is in:
processing.__init__
and comparing.__init__
@@ -10,10 +11,46 @@ Remove DUPLICATE: check_scene is in:
Move check_scene, is_numeric_array to:
processing.__init__
-
+------------------------------------------------------
+0002: Restructure processing:
Move function in processing/__init__ to processing/place_code.py
Move function optic_flow in processing/place_code.py to processing/motion_code.py
+------------------------------------------------------
+0003: Improve database
+In the init database I would like to use class properties instead of get/read
+line: 263,273,and 394
+ def create(self)
+should be replaced by:
+ @property
+ def create(self)
+
+It implies that at every point that self.create() is called should be changed to self.create [Note the absence of parenthesis]
+
+line: 298
+ def get_posorients(self)
+should be replaced by
+ @property
+ def posorients(self)
+
+Need to propagate the changes through all the code (see rendering / processing / moving )
+------------------------------------------------------
+0004: Change every assert by a if () raise TypeError/IOError/KeyError/...
+- present in processing
+- present in database
+
+------------------------------------------------------
+0005: Write test function for raise Error
+- for every raise error create a test function, checking that the error is correctly thrown (see moving/test_agent for inspiration)
+
+------------------------------------------------------
+0006: Improve comparing/__init__.py
+- Change comment in simple_image_diff
+- call simple_image_diff in imagediff. you can then remove all the assert (that should be raise Error at that point) from imagediff
+- Add comment to diff_optic_flow / add reference to article.
+- rename capitalised variable A,ATA, and b as longer variable name [future PEP8 will forbid this]
+------------------------------------------------------
+0007: Fix test processing
- Fix db in test function such that it work from any location. (probably need to add a module resources and function to load them)
- Test are failing WHY???
--
GitLab