Status Reports (Nov 9, 2012) - Review Board UCOSP

The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Message from discussion Status Reports (Nov 9, 2012)
Received-SPF: pass (google.com: domain of sampson.c...@gmail.com designates 209.85.214.175 as permitted sender) client-ip=209.85.214.175;
MIME-Version: 1.0
In-Reply-To: <CAER+-ePpVyFv_WtwrONPDHeqjqfCCUzpVh2gjQa=en_X3ZH...@mail.gmail.com>
References: <ca1b3552-72ff-477b-852d-e98837205e3b@googlegroups.com>
 <CAC3F7szq_TuxL+2p8te6QRSiR2M=2nJjqTa84rivV0h=N_G...@mail.gmail.com>
 <CANVpq6qj_G66_GGKnvh2eFjCGoOjUUxviGqK1aEjjUAdPui...@mail.gmail.com> <CAER+-ePpVyFv_WtwrONPDHeqjqfCCUzpVh2gjQa=en_X3ZH...@mail.gmail.com>
From: Sampson Chen <sampson.c...@gmail.com>
Date: Sat, 10 Nov 2012 12:29:10 -0500
Message-ID: <CAG9_39bFLpDP=2E=mEmOBAgNajDn8DRs-UFdSNE_m+PG+ku...@mail.gmail.com>
Subject: Re: Status Reports (Nov 9, 2012)
To: reviewboard-ucosp@googlegroups.com
Content-Type: multipart/alternative; boundary=e89a8fb2048038364604ce276908

--e89a8fb2048038364604ce276908
Content-Type: text/plain; charset=ISO-8859-1

*Sampson Chen*

*Currently: *
- Reading about how memcaching works in RB
- Started coding for Extension integration with ReviewUIs
- Been very busy this past week, but I should get uninterrupted time to
work on Sunday.

*Roadblocks:*
- Feels like I'm progressing very slowly with the extensions project. I've
read over the code in reviewboard/extensions, djblets/extensions,
reviewboard/review/ui, made notes as I went along, referred back to IRC
conversation history with mentors multiple times - but it's still fuzzy in
my head about exactly how to proceed.

*Next:*
- Get a WIP review-request for extensions project up ASAP.
- Add more polish to the thumbnails project (style issues)
- Possibly add memcaching to thumbnails project

*Questions:*
- None right now.

On Sat, Nov 10, 2012 at 11:11 AM, Allyshia Sewdat <allysh...@gmail.com>wrote:

> *Allyshia Sewdat
> *
>
> *Currently: *
> - Getting the non-WIP version of the JSLint plugin review request ready
> - Putting together mockups for the static analysis manual trigger UI
>
> *Roadblocks:
> *- Was held up by trying to figure out the best way to get JSLint to
> execute with custom options. Behaviour was unexpected whereby passing in
> select custom options would have JSLint stop sooner (scan less of the file)
> than passing in either no options or ALL options including those explicitly
> set to the same value as default. Experimented with online version and it
> does not seem to behave the same way. It would be a poor user experience to
> scan 18% of the file if they are expecting it to scan 75%. Trying to figure
> out why this is happening, but fallback solution that has been tested is to
> pass in explicit default values for all parameters which RB users do not
> configure via admin panel.
>
> *Next:*
> - Focus on second project : manual trigger.
>
> *Questions:*
> None
>
>
>
> On Fri, Nov 9, 2012 at 11:12 PM, Tina Yang <tinayang...@gmail.com> wrote:
>
>> *Tina Yang*
>>
>> *Currently: *
>> - Modify diff_file_fragment.html to display the binary files inline better
>>
>> *Roadblocks:*
>> None
>>
>> *Next:*
>> - Add unit tests and possibly more manual test cases
>>
>> *Questions:*
>> None
>>
>>
>> ---------------------------------------------------------------------------------------------------------
>>
>> John,
>> I'm not entirely sure, so please correct me if I'm wrong, but I see that
>> you can watch each group in your account settings under:
>> "Which groups do you belong to or wish to watch?"
>> I would assume that if you watch the reviewboard group if anyone submit a
>> new review request to the reviewboard repository you would get a
>> notification for it.
>>
>>
>> On Fri, Nov 9, 2012 at 7:54 PM, Aamir Mansoor <m...@aam1r.com> wrote:
>>
>>> *Aamir Mansoor*
>>>
>>> *Currently: *
>>> - Finished addressing issues in r/3434
>>>
>>> *Roadblocks:*
>>> - Not sure what to work on next for Markdown UI. Some ideas come to
>>> mind: work on the frontend for what I have done so far or move on to
>>> another file extension while my latest changes get reviewed.
>>>
>>> *Next:*
>>> - Figure out what to work on next for Pluggable UI
>>>
>>> *Questions:*
>>> None
>>>
>>> Aamir Mansoor
>>> http://aam1r.com<http://aam1r.com?utm_source=gmail&utm_medium=email_signature&utm_campaign=me%40aam1r.com>
>>>
>>>
>>>
>>> On Fri, Nov 9, 2012 at 9:18 PM, John Sintal <john.sin...@gmail.com>wrote:
>>>
>>>> *Reports are due 24hrs before meeting, I believe. So, have them in by
>>>> Sat 2pm and I shall post them soon after.*
>>>>
>>>> *John Sintal*
>>>>
>>>> *Currently: *
>>>> - Still working on rb patch. Almost done, it's just that last 5% that's
>>>> in my way.
>>>> - Figuring out how to properly testing rb patch and rb diff.
>>>>
>>>> *Roadblocks:*
>>>> - As said here<https://groups.google.com/forum/?fromgroups=#!topic/reviewboard-ucosp/piQjc9mWF08>,
>>>> I am having issues with executing the command in Python. The command works
>>>> fine otherwise for most cases.
>>>> - Proper pX argument. I believe ChipX86 said he had a friend who had
>>>> the logic on figuring out the best configuration.
>>>> - Testing could be challenging and impede my work on new commands. It's
>>>> been discussed by Steven that testing isn't trivial for this sort of stuff,
>>>> especially with all the SCMs and a lack of a testing structure for rb
>>>> commands. I forsee myself spending a lot of time on this as opposed to
>>>> working on new commands.
>>>> - I am finding code review to be tough since most the review requests
>>>> are already tackled by the mentors. I also wish there were more
>>>> opportunities to look for pep8 style violations, as those are easier
>>>> to detect. Perhaps it would be possible to let students tackle those types
>>>> of problems (mentors purposely leave them). Altho, my idea might just be
>>>> crazy and I just need to try harder in terms of picking new reviews.
>>>>
>>>> *Next:*
>>>> - Finish rb patch which includes the next two tasks:
>>>>     - Code clean up (decide with Steven on refactoring decisions, which
>>>> options to leave in/out, assess the options that are superfluous yet still
>>>> are required by legacy code)
>>>>     - Testing on many SCMs
>>>>     - Getting the cmd to actually execute in the Python script
>>>>     - Figure out the pX logic
>>>> - Find review requests to review.
>>>>
>>>> *Questions:*
>>>> - Is there an option to receive notification of anyone posting a
>>>> review? This way I can be first to look at the review.
>>>>
>>>
>>>
>>
>

--e89a8fb2048038364604ce276908
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

<div style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13=
px;background-color:rgb(255,255,255)"><b><u>Sampson Chen</u></b></div><div =
style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;ba=
ckground-color:rgb(255,255,255)">

<br></div><div style=3D"color:rgb(34,34,34);font-family:arial,sans-serif;fo=
nt-size:13px;background-color:rgb(255,255,255)"><div style=3D"font-size:12.=
727272033691406px"><b>Currently:=A0</b></div><div style=3D"font-size:12.727=
272033691406px">

- Reading about how memcaching works in RB</div><div style=3D"font-size:12.=
727272033691406px">- Started coding for Extension integration with ReviewUI=
s</div><div style=3D"font-size:12.727272033691406px">- Been very busy this =
past week, but I should get uninterrupted time to work on Sunday.</div>

<div style=3D"font-size:12.727272033691406px"><br></div><div style=3D"font-=
size:12.727272033691406px"><b>Roadblocks:</b></div><div style=3D"font-size:=
12.727272033691406px">- Feels like I&#39;m progressing very slowly with the=
 extensions project. I&#39;ve read over the code in reviewboard/extensions,=
 djblets/extensions, reviewboard/review/ui, made notes as I went along, ref=
erred back to IRC conversation history with mentors multiple times - but it=
&#39;s still fuzzy in my head about exactly how to proceed.</div>

<div style=3D"font-size:12.727272033691406px"><br></div><div style=3D"font-=
size:12.727272033691406px"><b>Next:</b></div><div style=3D"font-size:12.727=
272033691406px">- Get a WIP review-request for extensions project up ASAP.<=
/div>

<div style=3D"font-size:12.727272033691406px"><font color=3D"#222222" face=
=3D"arial, sans-serif">- Add more polish to the thumbnails project (style i=
ssues)</font></div><div style=3D"font-size:12.727272033691406px"><font colo=
r=3D"#222222" face=3D"arial, sans-serif">- Possibly add memcaching to thumb=
nails project</font></div>

<div style=3D"font-size:12.727272033691406px"><font color=3D"#222222" face=
=3D"arial, sans-serif"><br></font></div><div style=3D"font-size:12.72727203=
3691406px"><font color=3D"#222222" face=3D"arial, sans-serif"><b>Questions:=
</b></font></div>

<div style=3D"font-size:12.727272033691406px"><font color=3D"#222222" face=
=3D"arial, sans-serif">- None right now.</font></div></div><br><div class=
=3D"gmail_quote">On Sat, Nov 10, 2012 at 11:11 AM, Allyshia Sewdat <span di=
r=3D"ltr">&lt;<a href=3D"mailto:allysh...@gmail.com" target=3D"_blank">ally=
sh...@gmail.com</a>&gt;</span> wrote:<br>

<blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=
x #ccc solid;padding-left:1ex"><div><div style=3D"font-family:arial,sans-se=
rif;font-size:13px"><b><u>Allyshia Sewdat<br></u></b></div><div style=3D"fo=
nt-family:arial,sans-serif;font-size:13px">

<br></div><div style=3D"font-family:arial,sans-serif;font-size:13px">
<b>Currently:=A0</b></div>

<div style=3D"font-family:arial,sans-serif;font-size:13px">- Getting the no=
n-WIP version of the JSLint plugin review request ready<br>- Putting togeth=
er mockups for the static analysis manual trigger UI<br></div><div style=3D=
"font-family:arial,sans-serif;font-size:13px">


<br></div><div style=3D"font-family:arial,sans-serif;font-size:13px">

<b>Roadblocks:<br></b>- Was held up by trying to figure out the best way
 to get JSLint to execute with custom options. Behaviour was unexpected=20
whereby passing in select custom options would have JSLint stop sooner=20
(scan less of the file) than passing in either no options or ALL options
 including those explicitly set to the same value as default.=20
Experimented with online version and it does not seem to behave the same
 way. It would be a poor user experience to scan 18% of the file if they
 are expecting it to scan 75%. Trying to figure out why this is=20
happening, but fallback solution that has been tested is to pass in=20
explicit default values for all parameters which RB users do not=20
configure via admin panel.<br></div><div style=3D"font-family:arial,sans-se=
rif;font-size:13px"><br></div><div style=3D"font-family:arial,sans-serif;fo=
nt-size:13px"><b>Next:</b></div>

<div style=3D"font-family:arial,sans-serif;font-size:13px">- Focus on secon=
d project : manual trigger.<br></div><div style=3D"font-family:arial,sans-s=
erif;font-size:13px"><br></div><div style=3D"font-family:arial,sans-serif;f=
ont-size:13px">




<b>Questions:</b></div><div style=3D"font-family:arial,sans-serif;font-size=
:13px">None</div></div><div class=3D"HOEnZb"><div class=3D"h5"><div style=
=3D"font-family:arial,sans-serif;font-size:13px"><br></div><div class=3D"gm=
ail_extra">

<br><br><div class=3D"gmail_quote">
On Fri, Nov 9, 2012 at 11:12 PM, Tina Yang <span dir=3D"ltr">&lt;<a href=3D=
"mailto:tinayang...@gmail.com" target=3D"_blank">tinayang...@gmail.com</a>&=
gt;</span> wrote:<br><blockquote class=3D"gmail_quote" style=3D"margin:0 0 =
0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div style=3D"font-family:arial,sans-serif;font-size:13px"><b><u>Tina =
Yang</u></b></div><div style=3D"font-family:arial,sans-serif;font-size:13px=
"><br></div><div style=3D"font-family:arial,sans-serif;font-size:13px"><b>C=
urrently:=A0</b></div>




<div style=3D"font-family:arial,sans-serif;font-size:13px">-=A0Modify diff_=
file_fragment.html to display the binary files inline better</div><div styl=
e=3D"font-family:arial,sans-serif;font-size:13px"><br></div><div style=3D"f=
ont-family:arial,sans-serif;font-size:13px">




<b>Roadblocks:</b></div><div style=3D"font-family:arial,sans-serif;font-siz=
e:13px">None</div><div style=3D"font-family:arial,sans-serif;font-size:13px=
"><br></div><div style=3D"font-family:arial,sans-serif;font-size:13px"><b>N=
ext:</b></div>




<div style=3D"font-family:arial,sans-serif;font-size:13px">-=A0Add unit tes=
ts and possibly more manual test cases</div><div style=3D"font-family:arial=
,sans-serif;font-size:13px"><br></div><div style=3D"font-family:arial,sans-=
serif;font-size:13px">




<b>Questions:</b></div><div style=3D"font-family:arial,sans-serif;font-size=
:13px">None</div></div><div style=3D"font-family:arial,sans-serif;font-size=
:13px"><br></div><div style=3D"font-family:arial,sans-serif;font-size:13px"=
>



---------------------------------------------------------------------------=
------------------------------</div>
<div style=3D"font-family:arial,sans-serif;font-size:13px"><br></div><div s=
tyle=3D"font-family:arial,sans-serif;font-size:13px">John,=A0</div><div sty=
le=3D"font-family:arial,sans-serif;font-size:13px">I&#39;m not entirely sur=
e, so please correct me if I&#39;m wrong, but I see that you can watch each=
 group in your account settings under:</div>




<div><font face=3D"arial, sans-serif">&quot;Which groups do you belong to o=
r wish to watch?&quot;</font><br></div><div><font face=3D"arial, sans-serif=
">I would assume that if you watch the reviewboard group if anyone submit a=
 new review request to the reviewboard repository you would get a notificat=
ion for it.</font></div>


<div><div>

<div class=3D"gmail_extra"><br><br><div class=3D"gmail_quote">On Fri, Nov 9=
, 2012 at 7:54 PM, Aamir Mansoor <span dir=3D"ltr">&lt;<a href=3D"mailto:me=
@aam1r.com" target=3D"_blank">m...@aam1r.com</a>&gt;</span> wrote:<br><blockq=
uote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc =
solid;padding-left:1ex">




<div><b><u>Aamir Mansoor</u></b></div><div><br></div><div><div><b>Currently=
:=A0</b></div></div><div>- Finished addressing issues in r/3434</div><div><=
br></div><div><div><b>Roadblocks:</b></div></div><div>- Not sure what to wo=
rk on next for Markdown UI. Some ideas come to mind: work on the frontend f=
or what I have done so far or move on to another file extension while my la=
test changes get reviewed.</div>







<div><br></div><div><div><b>Next:</b></div></div><div>- Figure out what to =
work on next for Pluggable UI</div><div><br></div><div><b>Questions:</b></d=
iv><div>None</div><span><font color=3D"#888888"><div><br>

</div>Aamir Mansoor<div><a href=3D"http://aam1r.com?utm_source=3Dgmail&amp;=
utm_medium=3Demail_signature&amp;utm_campaign=3Dme%40aam1r.com" target=3D"_=
blank">http://aam1r.com</a></div></font></span><div><div>




<br>
<br><br><div class=3D"gmail_quote">On Fri, Nov 9, 2012 at 9:18 PM, John Sin=
tal <span dir=3D"ltr">&lt;<a href=3D"mailto:john.sin...@gmail.com" target=
=3D"_blank">john.sin...@gmail.com</a>&gt;</span> wrote:<br><blockquote clas=
s=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;pad=
ding-left:1ex">







<i>Reports are due 24hrs before meeting, I believe. So, have them in by Sat=
 2pm and I shall post them soon after.</i><div><br></div><div><b><u>John Si=
ntal</u></b></div><div><br></div><div><div style=3D"font-family:arial,sans-=
serif;font-size:12.727272033691406px">







<b>Currently:=A0</b></div><div style=3D"font-family:arial,sans-serif;font-s=
ize:12.727272033691406px">- Still working on rb patch. Almost done, it&#39;=
s just that last 5% that&#39;s in my way.</div><div style=3D"font-family:ar=
ial,sans-serif;font-size:12.727272033691406px">







- Figuring out how to properly testing rb patch and rb diff.</div><div styl=
e=3D"font-family:arial,sans-serif;font-size:12.727272033691406px"><br></div=
><div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px"=
>







<b>Roadblocks:</b></div><div style=3D"font-family:arial,sans-serif;font-siz=
e:12.727272033691406px">- As said <a href=3D"https://groups.google.com/foru=
m/?fromgroups=3D#!topic/reviewboard-ucosp/piQjc9mWF08" target=3D"_blank">he=
re</a>, I am having issues with executing the command in Python. The comman=
d works fine otherwise for most cases.</div>







<div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px">=
- Proper pX argument. I believe ChipX86 said he had a friend who had the lo=
gic on figuring out the best configuration.</div><div style=3D"font-family:=
arial,sans-serif;font-size:12.727272033691406px">







- Testing could be challenging and impede my work on new commands. It&#39;s=
 been discussed by Steven that testing isn&#39;t trivial for this sort of s=
tuff, especially with all the SCMs and a lack of a testing structure for rb=
 commands. I forsee myself spending a lot of time on this as opposed to wor=
king on new commands.</div>







<div><font face=3D"arial, sans-serif"><span style=3D"font-size:12.727272033=
691406px">- I am finding code review to be tough since most the review requ=
ests are already tackled by the mentors. I also wish there were more=A0</sp=
an>opportunities<span style=3D"font-size:12.727272033691406px">=A0to look f=
or pep8 style violations, as those are easier to detect. Perhaps it would b=
e possible to let students tackle those types of problems (mentors purposel=
y leave them). Altho, my idea might just be crazy and I just need to try ha=
rder in terms of picking new reviews.</span></font></div>







<div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px">=
<br></div><div style=3D"font-family:arial,sans-serif;font-size:12.727272033=
691406px"><b>Next:</b></div><div style=3D"font-family:arial,sans-serif;font=
-size:12.727272033691406px">







<font color=3D"#222222" face=3D"arial, sans-serif">- Finish rb patch which =
includes the next two tasks:</font></div><div style=3D"font-family:arial,sa=
ns-serif;font-size:12.727272033691406px"><font color=3D"#222222" face=3D"ar=
ial, sans-serif">=A0 =A0 - Code clean up (decide with Steven on refactoring=
 decisions, which options to leave in/out, assess the options that are supe=
rfluous yet still are required by legacy code)</font></div>







<div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px">=
<font color=3D"#222222" face=3D"arial, sans-serif">=A0 =A0 - Testing on man=
y SCMs</font></div><div style=3D"font-family:arial,sans-serif;font-size:12.=
727272033691406px">







=A0 =A0 - Getting the cmd to actually execute in the Python script</div><di=
v style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px"><fo=
nt color=3D"#222222" face=3D"arial, sans-serif">=A0 =A0 - Figure out the pX=
 logic=A0</font></div>







<div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px">=
<span style=3D"font-size:12.727272033691406px">- Find review requests to re=
view.</span><br></div><div style=3D"font-family:arial,sans-serif;font-size:=
12.727272033691406px">







<font color=3D"#222222" face=3D"arial, sans-serif"><br></font></div><div st=
yle=3D"font-family:arial,sans-serif;font-size:12.727272033691406px"><font c=
olor=3D"#222222" face=3D"arial, sans-serif"><b>Questions:</b></font></div><=
div style=3D"font-family:arial,sans-serif;font-size:12.727272033691406px">







<font color=3D"#222222" face=3D"arial, sans-serif">- Is there an option to =
receive notification of anyone posting a review? This way I can be first to=
 look at the review.</font></div></div></blockquote></div><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br>

--e89a8fb2048038364604ce276908--
Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
Account Options
