Re: [cclib-devel] New patch

[Noel O'B. <bao...@gm...>]

Great, but can you supply public domain test files for #1, #2, and #3?

- Noel

On 28 July 2012 22:55, Edward Holland <hol...@ca...> wrote:
> Hi all,
>
> I've made a few changes to cclib for some scripts i've been writing. These changes are
> 1) gaussian irc parser
> 2) gamess thermochemistry parser
> 3) refactored part of the gamess vibrations parser
> 4) Added some units to the converter
> 5) fixed some small errors
>
> The main change that might cause contention is number 3. The code in question was designed to skip a number of text lines, mostly relating to symmetry. The issue with this code is that it excepts a particular number of lines of text between each block. However this is not always the case with GAMESS output files (see bug #3476063). I found a number of other cases similar to those in bug #3476063 while working on the gamess parser. I found the most general, and only consistently valid, method of skipping these lines was to search for the start of the frequency analysis. The refactored code reflects this and passes all the cases in the tests folder.
>
> I hope the rest of the code is self explanatory.
>
> Yours
>
> Ed Holland
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> cclib-devel mailing list
> ccl...@li...
> https://lists.sourceforge.net/lists/listinfo/cclib-devel