DistRen Changeset - eb27dfe10342 · protofusion repositories

Changeset - eb27dfe10342

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

ethanzonca - 16 years ago 2009-07-25 18:30:12

Bugfixing, found some issues we need to address.

3 files changed with 5 insertions and 3 deletions:

TODO

src/server/slave.c

src/server/slavefuncs.c

0 comments (0 inline, 0 general)

TODO

➞

Show inline comments

 The most challenging and important tasks are marked with an "*"
+The most challenging and important tasks are marked with an "*", some items marked with initials to show who needs to fix them.
 Master
 *-Add calls to remotio
  -Write more meat into the pseudo-code main() to pull everything together
  -Write a stub for getting info from the tarball/validifying the tarball. Read distren-job.xml, a file in the tarball, to find out 1. which rendering system to use (that system, e.g. blender/povray, can read more specifics, such as name of file to pass to blender and frames. Options common between different systems will be handled in common as best as possible)
  -Write a stub for publishing file and constructing job description so that the job can be shared
 *-Other XML-writing/reading
 M-frame_finder() needs to return both the job number and the frame number
 Slave
  -Fix current bugs
  -Add conf-file editing code where commented in register_user()
 *-Add calls to remoteio, once it works or even before it works
 *-Figure out how to get output back to the server!
@@ @@ -16,7 +17,8 @@ Slave @@
 *-Other XML-writing/reading
 Options
  -Rewrite some stuff, try to make it simpler
  -Move server and client confuse code into their individual files, rather than in the common file.
  -Review all confuse interfacing
  -Push patch for relative includes in confuse to the confuse developers
@@ \ No newline at end of file @@
  -Push patch for relative includes in confuse to the confuse developers

src/server/slave.c

➞

Show inline comments

@@ @@ -111,12 +111,13 @@ options_init(argc,argv,&my_cfg, myopts, @@
 				delay(1000); // or not... this should be more event-driven, but should still give a heartbeat to the server
+			}
 		slavestatus=2; // When done rendering, set this.
+	}
 	if(slavestatus==2){
 		fprintf(stderr, "Finished frame %d in job %d",framenum,jobnum);
 		tell_the_server(DISTREN_FINISHED_ASSIGNMENT); // or implement this into the exec_X() functions, but that would be repetative
 		slavestatus=0;
+	}
 	if(slavestatus==0){
 		tell_the_server(DISTREN_REQUEST_DONEFRAME);
 		fprintf(stderr, "Idle. No frames to render.");
+	}

src/server/slavefuncs.c

➞

Show inline comments

@@ @@ -110,13 +110,12 @@ void exec_blender(struct blendjob* blend @@
+{
   char *frame_str;
   asprintf(frame,frame_str); // GNU/*nix compatible only, fix before releasing win32, although dll for windows for asprintf exists!
   int ret;
   char *cmd[] = { "blender", "-b", "-o", output, input, "-f", frame_str, (char *)0 };
   ret = execv("/usr/bin/blender", cmd); // This path will be absolute for testing, should be relative to install on production
   finish_frame(blendjob, frame); // Is this going to be client-side or server-side?
+}
 /* Update me when the time comes:
 void exec_luxrender(struct luxjob* luxjob, char *input, char *output, int frame)
+{
   char *frame_str;

0 comments (0 inline, 0 general)