Hi Jonathan,
So am I the original author at this point for this change? I was thinking you had made some subsequent changes and that took the ball away from me. 
But if not, I will be happy to do whatever is needed to fix it. 
 -dave

On Sep 13, 2013, at 10:28 AM, "Jonathan J. Helmus" <notifications@github.com> wrote:

> In aoslib/awips.py:
> 
> >         are replaced with flag value 1.e37 in the return array.
> >  
> >      Examples
> >      --------
> >      >>> import aoslib
> >      >>> aoslib.calctw([[1013.25]], [[296.15]], [[85.]])
> > -    array([[ 298.81143188]], dtype=float32)
> > +    array([[ 294.28710938]], dtype=float32)
> Green lines indicate additions to the file included with the Pull Request (PR), red line are those removed. I do not think anything needs to be changed, when I run the example code I get a value of 294.28710938, which is also in line with the values calculated using other methods.
> 
> In general if there are issues that need to be fixed in a PR before merging it is best to ask the original author to make them in his or her folk. When they make commit and push them to GitHub the PR will be automatically updated to include them until the PR is merged. If the original author is unable to unwilling to fix the issues, the changes can still be updated and merged without ever placing the repo in a "bad" state. Typically in these cases I checkout the PR fork, make the needed changes, and then merge into master locally. Pushing this to GitHub will close the PR automatically, but again the preferred method is to have the original author make the changes.
> 
> —
> Reply to this email directly or view it on GitHub.
>