Skip to content
This repository has been archived by the owner on Oct 17, 2020. It is now read-only.

Submit to CRAN #5

Open
antagomir opened this issue Mar 15, 2016 · 11 comments
Open

Submit to CRAN #5

antagomir opened this issue Mar 15, 2016 · 11 comments
Assignees
Milestone

Comments

@antagomir
Copy link
Member

Would be useful to take this to CRAN. I can do if there are no other volunteers. But I would still direct the emails to the main developer JJ. Any comments ?

@antagomir antagomir self-assigned this Mar 15, 2016
@antagomir antagomir added this to the CRAN milestone Mar 15, 2016
@statguy
Copy link
Member

statguy commented Mar 15, 2016

Since I have moved on to very different kind of projects, I would be happy hand this package over to anyone who wants to continue the work. Note that if you want to put this to CRAN, you also need to put the rwfs package.

@antagomir
Copy link
Member Author

I can't take over as this is not my core area and I maintain already >10 packages. But I can contribute by submitting to CRAN if we like to do that.

@statguy
Copy link
Member

statguy commented Mar 16, 2016

If doesn't require major involvement from me, then go ahead.

@jlehtoma
Copy link
Contributor

jlehtoma commented Jun 5, 2016

I would indeed very useful to update the package and push it to CRAN. @torsti might be interested in getting involved in the package maintenance, I can help out as well. Apparently the current version is no longer working the FMI API, so the package needs more work than making sure it passes CRAN checks.

@torsti
Copy link
Collaborator

torsti commented Jun 5, 2016

I'll try to take an initial look in the next week. I'd prefer to have a Python tool for my own FMI data needs. But if we get this working, maybe that would be enough.

@jlehtoma
Copy link
Contributor

jlehtoma commented Jun 5, 2016

👍 fantastic, let's have a look at this in case there's a quick fix (e.g. altered structures in the API).

BTW, for those interested in access through Python, there is also the FMI-weather-downloader GUI software for downloading the data over the FMI API. Not sure if it has an associated library, but in any case.

@torsti
Copy link
Collaborator

torsti commented Jun 5, 2016

The downloader is a good tool for the things it does, but I've more need for a framework to work with than a few prepared alternatives. Though it uses the lxml python library, which if i remember correctly is based on libxml2, so there might be something in their approach that we could use if we end up building new functionality on top of the xml2 package from CRAN.

@jlehtoma
Copy link
Contributor

jlehtoma commented Jun 5, 2016

As mentioned in issue #6, fmi (or actually rwfs) uses GDAL to read in the data, so no need (or hope) to parse it separately.

@torsti
Copy link
Collaborator

torsti commented Jun 5, 2016

Ok, so rfws needs to go to CRAN first.

@jlehtoma
Copy link
Contributor

jlehtoma commented Jun 5, 2016

Yep, I think so too. However, I think fixing fmi is a higher priority as it can always be installed from GitHub. But as for submitting it to CRAN, rwfs needs to go first.

@ouzor
Copy link
Contributor

ouzor commented Jun 5, 2016

Having rwfs in CRAN would be nice for gisfin package too, since quite a lot
of spatial data in Helsinki region is now made open via WFS APIs. I have
some code for those that use rwfs in a separate branch of gisfin. So +1 for
the plan!

On Sun, Jun 5, 2016 at 8:31 PM, Joona Lehtomäki notifications@github.com
wrote:

Yep, I think so too.


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

@jlehtoma jlehtoma added the ready label Sep 6, 2016
@jlehtoma jlehtoma modified the milestones: 0.3, CRAN Jan 29, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants