-
Notifications
You must be signed in to change notification settings - Fork 9
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
Work done primarily in build_rates (and related tests) #4
base: master
Are you sure you want to change the base?
Conversation
rynkwn
commented
Oct 3, 2016
- Updated rates.RData up to Sept. 16, 2016
- Made several fixes to build_rates.
- Removed a nil variable that was never assigned.
- Adjusted one of the if statements that relied on the nil variable.
- Added a test for build_rates to ensure that vectors aren't passed in for inputs. (Causes R session to crash).
- Adjusted some tests: No longer uses stopifnot(statement), now uses expect_true(statement)
…s a fatal crash. Modified boolean operator as we expect a single element and not a vector.
'test that rates don't move too much day-over-day' test is failing.
upfront_to_spread test improved. stopifnot statements converted to expect_true, showing more reasonably output.
Getting the following calling build_rates or download_markit: |
It looks like the problem is in get_raw_markit() function. The call is getting timed out if internal network requires proxy settings for the curlPerform to connect directly to www.markit.com. If I do the below proxy setting before the curlPerform the request goes through. Could you add the variable to the creditr package functions for the user to be able to specify proxy settings and test if that does the trick? curl <- getCurlHandle() a <- tryCatch(curlPerform(url = 'https://www.markit.com/news/InterestRates_USD_20180515.zip', |
Hey @sovboss, sorry, but it's been a long time since I worked with this code base. I'm not sure who's primarily working on the code base at this time, but it may be worth leaving an issue on this project (https://github.com/davidkane9/creditr/projects/1) if you can! (I'm not sure where the Issues section has gone for this Github repo.) |