[codeface] Re: [PATCH 00/26] Several fixes and enhancements for Codeface

  • From: Claus Hunsen <hunsen@xxxxxxxxxxxxxxxxx>
  • To: codeface@xxxxxxxxxxxxx
  • Date: Mon, 28 Nov 2016 16:22:18 +0100

Hi Wolfgang,

thank you very much for your comments. Better late than never. ;)

I have worked on most commits already, I will send the updated patch
stack later this week, hopefully. There are also 4 new commits incoming.

Regarding your comments, I will reply to all messages for which I have
responses of any kind. I guess, it's not necessary to reply to all
comments of yours. ;)

-- Claus

On 24/11/16 10:45, Wolfgang Mauerer wrote:

Hi Claus,

Am 13/10/2016 um 17:29 schrieb Claus Hunsen:
Hi all!

As requested by Wolfgang on GitHub [1], here is the current patch stack
on my 'claus-updates' branch.
[1] https://github.com/siemens/codeface/pull/51

Summarized, the changes are the following. A full list can be found below.
- move cppstats to most recent version (v0.9.2)
- some ducumentaion improvements
- some bugfixes
- add 'Base_Feature' for 'feature' analysis analogous to 'File_Level' in 
'proximity' analysis
- fix date parsing in ML analysis
- fix NULL authors in ML analysis (props to @bockthom)
- add messageId and creationDateOffset columns in mail table
- add --use--corpus option to CLI to force !use.mbox
- fix several bugs, warnings and imperfections in ML-analysis function
- fix the ML test
- improve logging
- fix TravisCI scripts

thanks for posting the patches in eMail form -- I have just a few
comments on some of them. Since this may be the latest patch review
in the history of software development, I'm very sorry for that.
Thanks heaps for your contributions!

Best regards, Wolfgang


Claus Hunsen (23):
  Fix inconsistent spacing in install_common.sh
  Move cppstats to most recent version (v0.9.1)
  Add __str__ for PersonInfo for better debugging
  Fix problem with feature blocks in a file from VCS
  Sort line information in FileDict
  Add 'Base_Feature' analogous to 'File_Level'
  Fix feature-related tests regarding 'Base_Feature'
  Add further documentation for the feature-related tests
  Move cppstats to most recent version (v0.9.2)
  Improve parsing of date headers in the ML analysis
  Further improve parsing of date headers in the ML analysis
  Re-parse all date header in the ML analysis
  Fix feature analysis
  Further improve date parsing in ML analysis
  Add 'messageId' column to DB table 'mail'
  Add 'creationDateOffset' column to DB table 'mail'
  Fix cycle extraction in ML analysis
  Fix the ML-analysis test
  Fix Travis problems with GnuPG and rJava
  Fix potential error with 'normalize' function in R
  Add option '--use-corpus' to ML analysis
  Fix ML test
  Prevent warnings in 'scale.data' if data is empty

Thomas Bock (3):
  Fix problem with NULL authors in mailing list analysis
  Fix number of messages in mailing list download
  Enhance log message for removed mail reference id lines

Attachment: signature.asc
Description: OpenPGP digital signature

Other related posts: