<div dir="ltr">Thank you!<div><br></div><div>James:</div><div>I take your point... I am going to make separated branches one for collaboration and another for just a SimpleActivity Activity.</div><div>I will try to do that today, anyway I will notify.</div>
<div><br></div><div>Regards,</div><div>aguz</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/22 James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<br>
I have reviewed simpleactivity.py and testactivity.py<br>
<br>
This code is very legible and explains well what it does, and can<br>
serve as an example for new activity authors.<br>
<br>
The docstrings in simpleactivity.py are an effective explanation of<br>
the simplified API, so perhaps you can generate the documentation from<br>
them in the usual Python fashion.<br>
<br>
Next to do is for a few activities to be developed using<br>
SimpleActivity, so that you can see what remains common to the new<br>
activities.  The common code might then be added to SimpleActivity.<br>
<br>
testactivity.py is derived from SharedActivity, and so it is complex,<br>
because a collaborating activity is complex.  I'd like to see also an<br>
activity example derived from SimpleActivity.<br>
<br>
The number of imports done by testactivity.py still seems high, and a<br>
SimpleActivity example may be able to reduce that.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span></blockquote></div><br></div>