[Bioc-devel] reverting to older version

Samsiddhi Bhattacharjee @b@@t@tgen @ending from gm@il@com
Mon Jun 11 18:27:09 CEST 2018


I am maintainer of package ASSET. We have recently discovered some issues
(most importantly computational speed issues) with recent versions of our
package and wanted to revert the code to an older version ASSET v 1.8.0
present in Bioconductor release 3.2, before proceeding to make further
enhancements to the package.

In release 3.3 , there were major changes to the package, it is like a
branch that we now realize that we need to abandon. We had introduced a new
feature and for that we switched from deterministic p-value calculation to
stochastic calculation. We did not notice the issues untill now. We want to
switch back to the deterministic one, which was present last in 3.2.

As suggested by Nitesh, I have made the changes in devel branch (basically
by copying the code as it was in release 3.2, and only updating the
DESCRIPTION file make the version 1.99.0 as this will be a major change
(although we are taking a few steps back, we will probably add some steps
forward before release 3.8).

I wanted to put a .onAttach() message in the current version to make the
user aware of the issues and possibly mentioning the next release and/or
pointing to the older release. However, as Herve
has pointed out, people may mix up devel and release versions causing
problems. Hence Herve had suggested:

"It will be much better if you actually fix the release version of your
package. This should just be a matter of porting the fixes you do in devel
with 'git cherry-pick'."

Reason I am hesitating is that the changes (diff of 3.7 and 3.2) are quite
a lot and doing selective changes as suggested will introduce further bugs,
and even after selection these changes will be *many*. Is it ok to backport
a "patch" to the release with a large number of changes? If yes, what
should the version number be bumped to?

Thanks in advance.



	[[alternative HTML version deleted]]

More information about the Bioc-devel mailing list