Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with flow history calculations when there are missing years #92

Closed
rmhirsch49 opened this issue Jul 20, 2016 · 4 comments
Closed

Comments

@rmhirsch49
Copy link

Laura: I found a bug, fairly serious, in the makeAnnualSeries function that has consequences for a variety of other functions like plotFlowSingle, printSeries, plotFourStats and plotFour. I worked at it and found that I was able to write a fix for it (all the changes are in the loop near the bottom that goes from istat in 1:8. I've attached my code, but please look at it and also test it yourself. A good site to test it on is 11336580 (has a lot of missing data). Probably should be tested on one with no missing data (my changes should be of no consequence there.

So, this needs to go up on CRAN and GRAN fairly soon. It is a bad bug. You can see what I mean when you run it through plotFourStats without fixing makeAnnualSeries.

Thanks.

Bob

I'll send the code as an R file by email. Funny that they don't let me attach an R file in GitHub.

makeAnnualSeriesFixed.txt

@ldecicco-USGS
Copy link
Collaborator

I should be able to get to this this afternoon, or tomorrow.

@ldecicco-USGS
Copy link
Collaborator

Pushing to GRAN and CRAN shortly.

@rmhirsch49
Copy link
Author

Super. Thanks. What version number will that have on it?

Bob

On Tue, Jul 26, 2016 at 12:30 PM, Laura DeCicco notifications@github.com
wrote:

Pushing to GRAN and CRAN shortly.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#92 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABHd0V61QXzJWKml9Mno4oTwt49vzSZNks5qZjYxgaJpZM4JRNN6
.


@ldecicco-USGS
Copy link
Collaborator

2.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants