From mboxrd@z Thu Jan 1 00:00:00 1970 From: Priya Subject: Re: Regarding Outreachy project on Improving CR Dashboard Date: Thu, 7 Apr 2016 17:57:33 +0530 Message-ID: References: <1459898587.7498.95.camel@bitergia.com> <1459979990.7498.156.camel@bitergia.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2980020992196302274==" Return-path: In-Reply-To: <1459979990.7498.156.camel@bitergia.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: "Jesus M. Gonzalez-Barahona" Cc: Lars Kurth , Xen-devel , Daniel Izquierdo List-Id: xen-devel@lists.xenproject.org --===============2980020992196302274== Content-Type: multipart/alternative; boundary=001a113aaf06ef3474052fe4337e --001a113aaf06ef3474052fe4337e Content-Type: text/plain; charset=UTF-8 Hello all, Thanks for the suggestions. I have updated the changes as u had mentioned. I am sorry, but I could not find any errors while running $ python3 createjson.py --mbox xen-devel-2016-03 --output new.json command. I am wondering what is wrong with this and might be a problem with python3 or my perceval version. I have added licensing and python logging. You can see it in my github repo [1]. I will try upgrading perceval and adding in the tests in the coming days, and will update. [1]: https://github.com/priya299/Dashboard *Priya V* Amrita University LinkedIn | GitHub | Bitbucket On Thu, Apr 7, 2016 at 3:29 AM, Jesus M. Gonzalez-Barahona wrote: > On Wed, 2016-04-06 at 17:30 +0530, Priya wrote: > > Hello, > > > > Thanks for your suggestions. > > I have made the appropriate changes as you had mentioned. > > It took a little time to change from python3 to python3.4 as perceval > > supports python3.4. I have updated the changes in my github. You can > > see my git repo [1] > > > > [1]:https://github.com/priya299/Dashboard > > Thanks a lot, Priya. Good work. Some preliminary comments, below. > > * When runing the script on the xen-devel-2016-03 mbox, I seen an > exception raised: > > ------------ > (perceval)jgb@expisito:~/src/outreachy/Dashboard/dashboard$ python3 > createjson.py --mbox xen-devel-2016-03 --output new.json > Traceback (most recent call last): > File "createjson.py", line 61, in > create_json(args.mbox,args.output) > File "createjson.py", line 43, in create_json > if key == k['Message-ID'].strip('<>'): > KeyError: 'Message-ID' > ------------ > > Maybe some message is not having a Message-ID field? I suggest that you > capture this exception, print out the offending message, and go on with > the next one. You can use the Python logging package for printing out > this kind of information (you can see how to use it in the Perceval > package itself). But see below. > > * Minor typo in the README: > > Instead of > > eg: python3.4 createjson --mbox xen-devel-2016-03 --output new.json > > it should be > > eg: python3.4 createjson.py --mbox xen-devel-2016-03 --output new.json > > * The files have no licensing info. If you agree, it could be GPLv3, as > is Perceval itself. For that, it would be enough that you mimic the > header in Perceval files in your Python files (of course, indicating > your authorship information). > > * Which version of Perceval are you using? Some weeks ago, the format > of the dictionary produced by Perceval for each message changed. Now > the actual fields of the message are in a data subdictionary. Please, > check that: the above exception with respect to the Message-ID key > could be because of this... Please, try to make it work with master > HEAD for Perceval (I don't expect any new major change in the next > days/weeks, and I'll try to warn you in case some happens). > > * Could you please write at least one unit test for your code? You can > see examples of the testing schema we use in the tests directory in > Perceval, but we use vanilla unittest (the Python package for tests). > At this stage I don't need that you produce a whole set of tests, only > one or two to show that you know how to write unit tests, please. > > Saludos, > > Jesus. > > > Priya V > > Amrita University > > LinkedIn | GitHub | Bitbucket > > > > _______________________________________________ > > Xen-devel mailing list > > Xen-devel@lists.xen.org > > http://lists.xen.org/xen-devel > -- > Bitergia: http://bitergia.com > /me at Twitter: https://twitter.com/jgbarah > > --001a113aaf06ef3474052fe4337e Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Hello all,

Thanks for the suggestio= ns. I have updated the changes as u had mentioned. I am sorry, but I could = not find any errors while running

$ python3 createjson.py --mbox xe= n-devel-2016-03 --output new.json

command. I am wondering what is w= rong with this and might be a problem with python3 or my perceval version. = I have added licensing and python logging. You can see it in my github repo= [1]. I will try upgrading perceval and adding in the tests in the coming d= ays, and will update.

[1]: https://github.com/priya299/Dashboard

Priya V
Amrita University


On Thu, Apr 7, 2016 at 3:29 AM, Jesus M. Gon= zalez-Barahona <jgb@bitergia.com> wrote:
On Wed, 2016-04-06 at 17:30 +0530, Priya wro= te:
> Hello,
>
> Thanks for your suggestions.
> I have made the appropriate changes as you had mentioned.
> It took a little time to change from python3 to python3.4 as perceval<= br> > supports python3.4. I have updated the changes in my github. You can > see my git repo [1]
>
> [1]:https://github.com/priya299/Dashboard

Thanks a lot, Priya. Good work. Some preliminary comments, below.
* When runing the script on the xen-devel-2016-03 mbox, I seen an
exception raised:

------------
(perceval)jgb@expisito:~/src/outreachy/Dashboard/dashboard$ python3
createjson.py --mbox xen-devel-2016-03 --output new.json
Traceback (most recent call last):
=C2=A0 File "createjson.py", line 61, in <module>
=C2=A0=C2=A0=C2=A0=C2=A0create_json(args.mbox,args.output)
=C2=A0 File "createjson.py", line 43, in create_json
=C2=A0=C2=A0=C2=A0=C2=A0if key =3D=3D k['Message-ID'].strip('&l= t;>'):
KeyError: 'Message-ID'
------------

Maybe some message is not having a Message-ID field? I suggest that you
capture this exception, print out the offending message, and go on with
the next one. You can use the Python logging package for printing out
this kind of information (you can see how to use it in the Perceval
package itself). But see below.

* Minor typo in the README:

Instead of=C2=A0

eg: python3.4 createjson --mbox xen-devel-2016-03 --output new.json

it should be

eg: python3.4 createjson.py --mbox xen-devel-2016-03 --output new.json

* The files have no licensing info. If you agree, it could be GPLv3, as
is Perceval itself. For that, it would be enough that you mimic the
header in Perceval files in your Python files (of course, indicating
your authorship information).

* Which version of Perceval are you using? Some weeks ago, the format
of the dictionary produced by Perceval for each message changed. Now
the =C2=A0actual fields of the message are in a data subdictionary. Please,=
check that: the above exception with respect to the Message-ID key
could be because of this... Please, try to make it work with master
HEAD for Perceval (I don't expect any new major change in the next
days/weeks, and I'll try to warn you in case some happens).

* Could you please write at least one unit test for your code? You can
see examples of the testing schema we use in the tests directory in
Perceval, but we use vanilla unittest (the Python package for tests).
At this stage I don't need that you produce a whole set of tests, only<= br> one or two to show that you know how to write unit tests, please.

Saludos,

=C2=A0 =C2=A0 =C2=A0 =C2=A0 Jesus.

> Priya V
> Amrita University
> LinkedIn | GitHub | Bitbucket
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org=
> http://lists.xen.org/xen-devel
--
Bitergia: http://bitergia.com
/me at Twitter: https://twitter.com/jgbarah


--001a113aaf06ef3474052fe4337e-- --===============2980020992196302274== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwOi8vbGlzdHMueGVuLm9y Zy94ZW4tZGV2ZWwK --===============2980020992196302274==--