Extra TODO and options

Author: MarkSturtevant

src/IFPainter.cpp

@@ -18,12 +18,18 @@ IFPainter::~IFPainter()
 
 void IFPainter::drawImage(int x, int y, int width, int height, std::string imgPath)
 {
+	// TODO (Michael): Along with writing this method, figure out the best way to represent images in code.
+	// Perhaps there's a better way to store images; reading in images multiple times is inefficient, though if we only need
+	// to read in the image once (which I believe is the case), then reading images from a file should also be fine.
 	// TODO: this
 }
 
 void IFPainter::drawText(int x, int y, int fontSize, int width, std::string text)
 {
 	// TODO: this
+	// keep in mind that we'll need options for bold, italics, etc.
+	// Try to make these methods as intuitive as possible in the idea of "we want text here, so we're confident that text will go here"
+	// Keep in mind text wrapping.
 }
 
 void IFPainter::drawLine(int x1, int y1, int x2, int y2, int width, cv::Scalar color)

src/Options.cpp

@@ -0,0 +1,11 @@
+#include "Options.h"
+
+Options::Options()
+{
+
+}
+
+Options::~Options()
+{
+
+}

src/Options.h

@@ -0,0 +1,21 @@
+#pragma once
+
+#include "pch.h"
+
+/*
+ * The Options class holds report-related parameters that can be specified by the user, such as
+ * specifications on which renders should be present to the screen, report dimensions, section dimensions,
+ * etc.
+ */
+
+// TODO: Task (Ally): Prepare this class.  There's a lot of freedom to how you can build this class.  You can
+// mix in constant variables along with modifiable ones, and allow certain parameters to be set via the user.
+// you should code some way to set and retrieve the values of certain parameters.  You can use a map and/or
+// add a bunch of variables manually; it's up to you.
+
+class Options
+{
+public:
+	Options();
+	~Options();
+};

src/main.cpp

@@ -16,7 +16,12 @@ int main(int argc, char **argv) {
  */
 void readParameters(int argc, char** argv)
 {
-    // TODO: this
+    // TODO (Ally): Write this function to instantiate the options so it can be used in generateReport.
+
+    // TODO (Ally): This includes reading in the filepath of the file and storing it somewhere.
+
+    // TODO (Ally): Along with storing the filepath, you should give some sort of option/parameter for generating
+    // reports for an entire folder of models!
 
     // i.e. if user specifies to open the file in a gui, we'll put that here...
 
@@ -31,6 +36,8 @@ void readParameters(int argc, char** argv)
  */
 void generateReport()
 {
+    // TODO (Ally): Incorporate the Options into this method, and set the correct bounds on IFPainter.
+    
     // create image frame
     IFPainter img(1500, 1000);
 

src/pch.h

@@ -11,6 +11,7 @@
 #include <string>
 
 // Visualization project header files
+#include "Options.h"
 #include "IFPainter.h"
 #include "InformationGatherer.h"
 #include "PerspectiveGatherer.h"