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

Work done primarily in build_rates (and related tests) #4

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

rynkwn
Copy link
Collaborator

@rynkwn 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)

@sovboss
Copy link

sovboss commented May 16, 2018

Getting the following calling build_rates or download_markit:
Error in UseMethod("xmlSApply") :
no applicable method for 'xmlSApply' applied to an object of class "character"

@sovboss
Copy link

sovboss commented May 16, 2018

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()
curlSetOpt(.opts = list(proxy = "

:"), curl = curl)

a <- tryCatch(curlPerform(url = 'https://www.markit.com/news/InterestRates_USD_20180515.zip',
writedata = f@ref,
noprogress = TRUE,
verbose = FALSE,
ssl.verifypeer = FALSE,
curl = curl),
error = function(e){ return("Rates data not available at markit.com") })

@rynkwn
Copy link
Collaborator Author

rynkwn commented May 20, 2018

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.)

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

Successfully merging this pull request may close these issues.

2 participants