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

Test Eurostat Example #14

Merged
merged 8 commits into from
Jul 18, 2014
Merged

Test Eurostat Example #14

merged 8 commits into from
Jul 18, 2014

Conversation

Tungurahua
Copy link

Added an example of Eurostat data to the inst/data. As Eurostat provides SDMX2.0 only via SOAP I added the file locally. REST is only available for SDMX 2.1. The file is used from the readSDMX examples

In addition I noted that moving the documentation to Roxygen is on the issues list, so I did this change for readSDMX.R

Any comments welcome.

@briatte
Copy link

briatte commented Jul 17, 2014

Will check it out as soon as I come back to France!

Also, I discovered this related package yesterday.

@eblondel
Copy link
Member

For the Roxygen, i prefer not mixing it with the Eurostat dataset testing. I didn't have look yet enough in Roxygen, and if we move the Rdoc with it it will be probably later. At now, the Rdoc will not be generated automatically with Roxygen annotations. It would be good if you could revert or remove the commits on this, and update the ticket only targeting Eurostat test. We can further discuss the "oxygenization" in the ticket i've created.

  • what you want here is to provide a sample dataset only to make it available for the R doc examples. I'm fine with this, but at now consider to add it directly in the Rdoc file.
  • did you find any error when applying readSDMX? Initially, i've set up the inst/data mainly to use as test resources for each test. I think it would be useful to write a test for this new sample, and if it fails see what's wrong in order to fix the bugs.

@antagomir
Copy link

Hi, very interesting. I should check in more detail the connection to our eurostat R package which is in beta phase now. As far as I understand the rsdmx is a generic interface package which just gives an example with eurostat data but is more generally applicable. We might be able to utilize this more in the eurostat package and avoid unnecessary overlap, merge the efforts where useful.

@eblondel
Copy link
Member

Hello @antagomir , yes indeed the rsdmx intends to be a generic interface to read SDMX-ML documents. Testing it with EuroStat data was introduced with this request. Until now the rsdmx has been successfully tested and used on FAO & OECD SDMX web-services. Indeed it would be great if you consider its use in packages such as eurostat. Your comments & contributions are also welcome.

As suggested here, the Eurostat example could added as code sample for the Rdoc, and probably a unit test will be based on it.

@Tungurahua
Copy link
Author

Following @eblondel comment I rolled back the roxygen documentation so reading Eurostat SDMX is the main topic of this branch.

The documentation of Eurostat webservices is here. However, I am unsure if an implementation of the webservices should be part of rsdmx. What do you think?

@Tungurahua
Copy link
Author

@eblondel regarding errors applying readSDMX - there have been none. The data is parsed as expected. I will look into formalizing this as a test. However, I need to familiarize with teststhatfirst.

@eblondel
Copy link
Member

Good, however it seems the travis build failed for the package...
Can you do the following:

  • add <?xml version="1.0" encoding="UTF-8"?> starting the xml file
  • there is a typo in the example code (misssing "S" of "SDMX" in xml filename)
  • in the example readSDMX, can you replace F by FALSE (it seems Travis build doesn't like it)

For the test, if you want, i can add it, no problem

@eblondel eblondel added this to the 0.3 milestone Jul 17, 2014
@eblondel eblondel changed the title ReadSDMX Roxygen and Eurostat Example Test Eurostat Example Jul 17, 2014
@Tungurahua
Copy link
Author

Done. Thanks for pointing out the issues.

I also added a test for the Eurostat example by running the parsed results against a dputof the expected dataframe. I don't know if that's a sensible way of doing things. Any suggestions for improvement are welcome.

eblondel added a commit that referenced this pull request Jul 18, 2014
@eblondel eblondel merged commit de1e4c8 into opensdmx:master Jul 18, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants