rgcca-factory / RGCCA

https://rgcca-factory.github.io/RGCCA/
10 stars 11 forks source link

Contributing to the package #24

Closed llrs closed 2 years ago

llrs commented 3 years ago

Hi, many thanks for providing RGCCA it has been very useful for my thesis.

A user opened an issue on a package that depends on RGCCA and I noticed there is a branch aiming for a new CRAN release.

I have made some modifications to RGCCA from the CRAN version 2.1.2 on a fork and perhaps you are interested in some of those to be included.

Some of them are related to the infrastructure:

Changes relate to code efficiency and simplicity:

Some "upgrades":

I do not know the extent of the changes but a user complained that the new CRAN branch doesn't export previously exported functions like scale2. It would be nice if you could explain the extend of the changes and provide a release cycle (first .Deprecate the function then in next version mark it as .Defunct) it would help all, users and developers.

For instance I saw that on #23 there is a PR using the version of gliomaData package I created on Github to make it easier to test and work with the package.

Thanks and let me know if you want some pull requests.

llrs commented 3 years ago

Any update on this?

GFabien commented 3 years ago

Hi @llrs! Sorry for the extremely late answer...

First we are really happy that you find the package useful!

If you're willing to contribute that's great and we would be pleased to integrate your propositions. As you have probably seen, the active branch is the CRAN one and it brings a lot of changes compared to previous versions. Some things have already been simplified but there is definitely room to improve and simplify the code further.

About the changes that were made, as I jumped in this year I don't know either the extent but it would be definitely a good idea to publish a release note to explain them.

If you want to discuss further you can send an email to @Tenenhaus and I and we can arrange a meeting.

llrs commented 3 years ago

Sorry @GFabien I don't have your email but if you are interested I will email @Tenenhaus again (I already had exchanged some emails regarding my modifications on RGCCA with him on 2019).

GFabien commented 3 years ago

Sure, you can write to @Tenenhaus and he will add me to the loop.

llrs commented 3 years ago

@GFabien I sent an email to @Tenenhaus, hope to meet you soon