Meeting Minutes for Nov 11th, 2012


  • We have 3 more official meetings before we wrap!

Questions / Answers / Tips:

Q: (Karl-L) Should we make sure to plug the UCOSP program in our one required blog post?

A: The goal of the blog post is not really to plug the program so much. The stated goal is to show people outside of the program, including prospective students, projects, and sponsors, what it’s like inside the program. Just tell people what you’ve experienced so far, what you’ve been running into, etc. So really say whatever you want about it. 😉

Q: (John) ChipX86 said he was going to ask a friend for pX argument logic

A:  (ChipX86) The pX logic doesn’t actually exist. I was misunderstanding what he did. So you’ll have to compute based on the RepositoryInfo’s base/root path, current directory, and base path on the ReviewRequest from the server

Q: (purple_cow)  re: rbtools commands: in the diffs there are a bunch of commandline args that don’t apply why are those still there?

A: The SCM requires that they exist.

F/U: (purple_cow)That’s something you (John) should probably put some focus into, refactoring things so you don’t need to require those arguments.

Q: (Allyshia) I ran into a weird bug with ReviewBot which slowed down my review posting last week (I sent a message to Steven about it, so we can talk offline later). It doesn’t interrupt my plugin’s functionality but it does make it output less than it could. I’m going to post the new proposed fix (that might be temporary) to a review request right after this meeting and my goal is to focus a lot more on my larger project for the rest of the time. I figure that it’s best to leave the plugin with room left to polish but contribute more to a more interesting project. Does that sound right to you?

A: That sounds fine with me. You’ve taken the JSLint plugin to a good point so far, and you really are running low on time for the other project

Q: (aam1r) I am not sure what to work on next

A: “File attachment review UIs in a lightbox” – there’s a few things in ReviewUI and in the template that are the beginnings of that, but nothing is at all hooked up. Applies to anything that specifies it can be reviewed “inline”. Images being a good start. Ideally, we can flip a flag on the ReviewUI subclass and get a lightbox. There wouldn’t be any diffing. So it’s not intended as being for binary files uploaded with a diff, which later we’d want optional diffing abilities for. We may decide we want that (like md) inline, and it’s good for testing. The ReviewUI subclass shouldn’t have to do really anything different to be in a lightbox. This is more about the general infrastructure around ReviewUIs.

Q: (yangtina) When is the hard pencils down deadline for this term?

A: I believe we’d settled on a soft pencils down around December 4th. The hard pencils down will likely be when the steering committee asks for evaluations to be due. It will be likely a few days after December 4th.

Q: (tahnok) smacleod, is there some way to have settings fields that show up depending on an option?

A: I doubt that will be possible with the current system. You might have to try and use a single form field that incorporates all of those options, or just tell the user certain options are ignored. (To give everyone a better idea, Review Bot tools can specify an option attribute on their class which is used to create a form on the RB side. RB doesn’t actually know anything about the class though, so this options attribute is a dictionary specifying django form fields as strings. It gets serialized and sent across to be stored in the DB)

Q: (slchen) are there projects we can work on if we are interested in sticking around after UCOSP? I’m finding that I’m doing more learning here than in the classroom b/c it’s almost all exploratory. Having mentor feedback helps a ton.

A: There is always work. *always* and sticking around after the term ends is highly appreciated / awesome. 🙂

Other Notes:

  • ChipX86 is going to be especially busy for the next few weeks
  • Students, watch the calendar carefullyover these next couple of weeks. Don’t get stuck. Don’t forget to ask questions. Get unstuck.
  • m_conley will get a firm pencils down date from the steering committee and send it out via email
  • We’d be absolutely happy to have you all continue hacking on RB after the course ends. We’re growing an awesome community here, and it rocks when people stick around. Plus, we’re going to have another crop of students come January who are going to need help from people who know their way around the code / process.
  • When you’re a volunteer contributor, you just do your best when you have the time. 

Leave a Reply

Fill in your details below or click an icon to log in: Logo

You are commenting using your account. Log Out /  Change )

Google+ photo

You are commenting using your Google+ account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )


Connecting to %s