Re: [R-pkg-devel] Package submission to CRAN not passing incoming checks

2024-04-22 Thread Kasper Daniel Hansen
1. The version number complaint is the use of 0.1.2-9033. It is complaining about 9033. Personally, I would not use the - style, but that is a personal choice I guess. R-exts says "The mandatory ‘Version’ field gives the version of the package. This is a sequence of at least *two* (and

Re: [Bioc-devel] Important Bioconductor Release Deadlines

2024-04-22 Thread Kasper Daniel Hansen
It's a bit late, but I'll point out that the use of R-devel dates back 20y to the inception of the Bioconductor project. So this is the way it has always been. Of course, this is an argument of conservatism which is always dangerous, But at least it illustrates that this is not a small change to

Re: [Bioc-devel] Check time on macOS (MIRit)

2024-02-07 Thread Kasper Daniel Hansen
To be a bit more clear and vague at the same time: currently, some of our macOS builders exhibit weird slowdowns when the build system is run automatically compared to running builds "by hand". This is pretty weird, and hopefully we can figure out the cause and fix it. Specifically, builds on

Re: [Bioc-devel] build reports for apple silicon

2024-01-24 Thread Kasper Daniel Hansen
. > > > Lori Shepherd - Kern > > Bioconductor Core Team > > Roswell Park Comprehensive Cancer Center > > Department of Biostatistics & Bioinformatics > > Elm & Carlton Streets > > Buffalo, New York 14263 > ---------- > *From:* Bio

[Bioc-devel] build reports for apple silicon

2024-01-24 Thread Kasper Daniel Hansen
We're clearly producing binary builds for Apple Silicon. Why do we not have a build report for this platform? Best, Kasper [[alternative HTML version deleted]] ___ Bioc-devel@r-project.org mailing list

Re: [Bioc-devel] How to declare optional system requirement for package

2023-11-30 Thread Kasper Daniel Hansen
To me, whether or not this is a good idea depends a bit on the degree to which these functions are generally useful. Best, Kasper On Wed, Nov 29, 2023 at 5:41 PM Hervé Pagès wrote: > Hi Alex, > > On 11/28/23 22:47, Alex Wong via Bioc-devel wrote: > > > Hi there, > > > > I am the author of

Re: [Bioc-devel] Is my package suitable for Bioconductor?

2023-10-24 Thread Kasper Daniel Hansen
I am happy to talk on Zoom about this, since I have some interest in this work. Best, Kasper On Tue, Oct 24, 2023 at 9:21 AM Vincent Carey wrote: > agreedthis discussion is best discussed in the open > > On Tue, Oct 24, 2023 at 8:57 AM Lluís Revilla > wrote: > > > Hi Tim, > > > > I think

Re: [Bioc-devel] Python module "tensorflow_probability" not found

2023-07-06 Thread Kasper Daniel Hansen
> builds > > > and runs fine both on GitHub Actions (GitHub repo here: > > > https://github.com/fmicompbio/orthos) and on the Single Package > Builder. > > > We have also tested (locally) that it will use GPUs if available, and > > that > > > the GPU config

Re: [Bioc-devel] Python module "tensorflow_probability" not found

2023-07-05 Thread Kasper Daniel Hansen
So I think Kim is interfacing to tensorflow by using the keras package from CRAN (partly authored by the Rstudio people). This package leaves it to the user to install tensorflow, which is a highly non-trivial installation task. There is some partly helpful instructions for using conda together

Re: [Rd] codetools wrongly complains about lazy evaluation in S4 methods

2023-06-13 Thread Kasper Daniel Hansen
On Sat, Jun 3, 2023 at 11:51 AM Mikael Jagan wrote: > The formals of the newly generic 'qr.X' are inherited from the non-generic > function in the base namespace. Notably, the inherited default value of > formal argument 'ncol' relies on lazy evaluation: > > > formals(qr.X)[["ncol"]] >

Re: [Bioc-devel] BiocManager::install

2023-05-11 Thread Kasper Daniel Hansen
build upon BiocGenerics, IRanges etc. to > submit them to CRAN, to increase potential user base (b/c installation from > Bioconductor can be such a pain). And that’s really not the place I want to > be. > > Thanks and best wishes > Wolfgang > > > > > > > I

Re: [Bioc-devel] BiocManager::install

2023-05-10 Thread Kasper Daniel Hansen
Could we get a list of use cases from Wolfgang? I am confused about what the issue is. Is the issue that it is painful to work with R-devel in the "off" 6-months? If so, I agree that it should be easier (even if we don't recommend it). But I am having a hard time parsing the email. I can

Re: [Bioc-devel] Changing Maintainer Email Addresses

2023-05-01 Thread Kasper Daniel Hansen
You should change the maintainer email in the DESCRIPTION file and commit it to Bioconductor git. Make sure you're subscribed to the devel email list on the new email. Best, Kasper On Mon, May 1, 2023 at 11:02 AM Augustin Luna wrote: > I am currently the maintainer of > >

Re: [Bioc-devel] Question on tasks after successfully building a package

2022-03-30 Thread Kasper Daniel Hansen
You need to increase your package version number to trigger a new build on the submission package. On Tue, Mar 29, 2022 at 7:03 AM Kern, Lori wrote: > You can modify your package at any time. Please remember to push to > git.bioconductor.org with any updates and fix any ERROR or Warnings that

Re: [Bioc-devel] MAGMA executable

2021-12-13 Thread Kasper Daniel Hansen
Ignoring the license issues (which may be significant), I strongly dislike this installation strategy. It (IMO) unreasonable that you potentially write in system locations on package load. You're looking in /usr/local/bin R.home/bin <- this makes not sense, this is the R home location, why

Re: [Bioc-devel] Use set.seed inside function

2021-11-30 Thread Kasper Daniel Hansen
Martin's suggestion is the way to go. For display purposes, you basically never want random colors, you almost always want to select distinct colors and there are principled ways of doing that. Best, Kasper On Tue, Nov 30, 2021 at 6:59 AM Martin Morgan wrote: > Check out grDevices::hcl.pals

Re: [Bioc-devel] bpparam Non-deterministic Default

2021-11-29 Thread Kasper Daniel Hansen
This should be solved by the vignette appropriately calling set.seed and explaining why they set the seed and why it is not done automatically. In a visible code chunk. On Sat, Nov 27, 2021 at 7:31 AM Spencer Nystrom wrote: > This GitHub issue, although lengthy, discusses some of the technical

Re: [Bioc-devel] Issue with change in random sampling

2021-09-24 Thread Kasper Daniel Hansen
The random number generator is tracked by a global state, which is a violation of functional programming. This is unavoidable. This is why we really don't want packages to EVER touch the random number state for example by setting the seed. You can also be affected by this if any function you

[Bioc-devel] lessons learned: negative experiences with git log

2021-08-13 Thread Kasper Daniel Hansen
had changed. However, running git log --name-status I got this START git log == commit 955e30b3ae316265b4f5f130ab6f12c33081da54 Author: Kasper Daniel Hansen Date: Wed Aug 11 08:52:11 2021 -0400 Fixing length condition error :100644 100644 25e24de 3aee690 MDESCRIPTION

Re: [Bioc-devel] BiocCheck and version requirement

2021-04-08 Thread Kasper Daniel Hansen
> Bioconductor Core Team > > > > Roswell Park Comprehensive Cancer Center > > > > Department of Biostatistics & Bioinformatics > > > > Elm & Carlton Streets > > > > Buffalo, New York 14263 > > > > > > From: Kasper D

Re: [Bioc-devel] BiocCheck and version requirement

2021-04-08 Thread Kasper Daniel Hansen
conductor Core Team > > Roswell Park Comprehensive Cancer Center > > Department of Biostatistics & Bioinformatics > > Elm & Carlton Streets > > Buffalo, New York 14263 > ---------- > *From:* Bioc-devel on behalf of Kasper > Daniel Han

Re: [Bioc-devel] Suppressing messages from Rprofile

2021-04-08 Thread Kasper Daniel Hansen
You could also preprend your cat so it outputs something like INCLUDE_DIR=.. and then use grep and sed to trim it. Of course that can also end up being brittle. On Wed, Apr 7, 2021 at 10:48 PM Henrik Bengtsson wrote: > Correcting: the env var is R_LIBS_USER and nothing else. > > /Henrik > >

[Bioc-devel] BiocCheck and version requirement

2021-04-08 Thread Kasper Daniel Hansen
The latest BiocCheck (well, it may have been around for a bit) _requires_ the package to have a R >= 4.1 dependency. That seems new to me. Right now it's a bit irritating, because if you're submitting a package that works with latest stable release, you are now prohibited from installing it from

Re: [Rd] custom allocators, Valgrind and uninitialized memory

2021-03-30 Thread Kasper Daniel Hansen
On Tue, Mar 30, 2021 at 9:39 AM Tomas Kalibera wrote: > appropriate to tell valgrind about it. As this is becoming rather too > technical and specific to the internals, feel free to take this offline > with Simon and me. > Respectfully, this seems to me to be exactly the kind of exchange

Re: [Bioc-devel] Moving minfi classes definition to a lighter package

2021-03-03 Thread Kasper Daniel Hansen
I am happy to engage in a discussion about this, although I'm not sure that I am ultimately interested in having two packages. But first I would like to look at some dependency graphs. I am wondering what makes the dependency tree this big (and my tree is smaller than yours, but still big:

Re: [Bioc-devel] How to integrate code from a package not available on bioconductor

2021-02-11 Thread Kasper Daniel Hansen
This question is actually pretty complicated and involves matters of copyright and academic credit, which are two different things. In general you cannot take code from the internet and republish it under your own name. It sounds like you want to include something substantial (as opposed to say a

Re: [Bioc-devel] Compiling a cpp source code while installing package

2020-11-08 Thread Kasper Daniel Hansen
Alexandra, What you're proposing is possible, but it is definitely a non-trivial task, because you (as the package author) are now responsible for getting MAGMA compiled on Linux / Windows and OS X. If MAGMA is not designed as a library, that might be hard. Not impossible but hard. Best, Kasper

Re: [Bioc-devel] BiocManager and R-devel

2020-10-30 Thread Kasper Daniel Hansen
iocVersion > package. I'd recommend the one-library-one-Bioconductor version approach to > for managing multiple versions outlined in the vignette. Your solution > means that you cannot switch back to your previously working version, which > might have been important for reproducibility or sanity. &g

[Rd] bug report for make check

2020-10-30 Thread Kasper Daniel Hansen
I would like to request access to bugzilla to file a bug report on make check for R-devel. Following changes to all.equal.POSIXt, make check now reports an error if the environment variable TZ is set to TZ="US/Eastern" (and likely other values). This can be addressed by using the argument

Re: [Rd] timezone tests and R-devel

2020-10-30 Thread Kasper Daniel Hansen
On Fri, Oct 23, 2020 at 11:10 AM Sebastian Meyer wrote: > Yes, you are absolutely right and I'm pretty sure this will be fixed in > one way or another. > > IMO, the failing test should simply use all.equal.POSIXt's new argument > check.tzone=FALSE. > I agree that this is a simple fix and I am

[Bioc-devel] BiocManager and R-devel

2020-10-30 Thread Kasper Daniel Hansen
Installed R-devel as of today. I have a previous version of Bioconductor installed in site-library. I then do install.packages("BiocManager") # installs version 1.30.10 > library(BiocManager) Bioconductor version '3.11' is out-of-date; the current release version '3.12' is available with R

Re: [Bioc-devel] Package update before october 23rd deadline and build errors [duplicated, wrong email address previous]

2020-10-26 Thread Kasper Daniel Hansen
Giula, Provided your package has been accepted by a reviewer, it will be released as part of Bioconductor 3.12, as long as it satisfies the other requirements, which includes some deadlines for when it is supposed to pass check without errors. In general, around release, there are a lot of

Re: [Rd] timezone tests and R-devel

2020-10-23 Thread Kasper Daniel Hansen
an error. Best, Kasper On Fri, Oct 2, 2020 at 11:28 AM Kasper Daniel Hansen < kasperdanielhan...@gmail.com> wrote: > Yes, the potential issue I see is that > make check > fails when I explicitly set TZ. However, I set it to be the same as what > the system reports when I

Re: [Bioc-devel] VERSO issue #1689

2020-10-19 Thread Kasper Daniel Hansen
I can't speak for the reviewer, but the package currently has 2 WARNINGS. If I were you, I would fix those warnings and push the fixes to Github, at which point the package will be re-built and re-checked. You're not supposed to have any ERRORS / WARNINGS when you submit. I would also look at the

Re: [Rd] timezone tests and R-devel

2020-10-02 Thread Kasper Daniel Hansen
w. Even with old R 3.6.3, I see > > > R-3.6.3 --vanilla --slave -e 'attr(as.POSIXlt(Sys.time()), "tzone")' > [1] "" "CET" "CEST" > > > TZ='Europe/Berlin' R-3.6.3 --vanilla --slave -e > 'attr(as.POSIXlt(Sys.time()), "tzone")

[Rd] timezone tests and R-devel

2020-10-01 Thread Kasper Daniel Hansen
The return value of Sys.time() today with a timezone of US/Eastern is unchanged between 4.0.3-patched and devel, but on devel the following test fails all.equal(x, as.POSIXlt(x)) with x = Sys.time() This means that devel does not complete make tests (failure on tests/reg-tests-2.R) It is

Re: [Bioc-devel] BSgenome changes

2020-08-20 Thread Kasper Daniel Hansen
hes > > Leonard > > > On Tue, Aug 18, 2020 at 10:06 AM Hervé Pagès wrote: > >> On 8/18/20 01:40, Kasper Daniel Hansen wrote: >> > In light of this, could we get a version of GRCh37 with only a single >> > mitochondrial genome? >> >> You mean a BSgen

Re: [Bioc-devel] BSgenome changes

2020-08-18 Thread Kasper Daniel Hansen
In light of this, could we get a version of GRCh37 with only a single mitochondrial genome? On Fri, Aug 14, 2020 at 6:17 PM Hervé Pagès wrote: > Hi Felix, > > On 8/13/20 21:43, Felix Ernst wrote: > > Hi Leonard, Hi Herve, > > > > I followed your conversation, since I have noticed the same

Re: [Bioc-devel] LazyData in DESCRIPTION file

2020-05-01 Thread Kasper Daniel Hansen
Dept. for Applied Bioinformatics > Inst. for Cell Biology and Neuroscience > Goethe University Frankfurt > > Biologicum, Room 3.209 > Phone +49 (0)69/798-42118 > > On 30. Apr 2020, at 16:02, Kasper Daniel Hansen < > kasperdanielhan...@gmail.com> wrote: > > This

Re: [Bioc-devel] LazyData in DESCRIPTION file

2020-04-30 Thread Kasper Daniel Hansen
> > Dr. Vinh Tran > > Dept. for Applied Bioinformatics > Inst. for Cell Biology and Neuroscience > Goethe University Frankfurt > > Biologicum, Room 3.209 > Phone +49 (0)69/798-42118 > > On 30. Apr 2020, at 14:55, Kasper Daniel H

Re: [Bioc-devel] LazyData in DESCRIPTION file

2020-04-30 Thread Kasper Daniel Hansen
If you don't have lazy data on, you should be able to load the data by using data(). Let us say you example data is called expData. With lazy data print(expData) Without lazy data you need explicit loading data(expData) print(expData) For the purpose of examples, there is also a

Re: [Bioc-devel] HDF5Array failure on windows

2020-04-07 Thread Kasper Daniel Hansen
rspective) is usually to simply omit the [] > and let the R help system resolve the link dynamically (sometimes prompting > the user to choose, if there multiple man pages). > > > > Martin Morgan > > > > > > On 4/6/20, 9:55 AM, "Bioc-devel on behalf of Kasper Daniel Han

[Bioc-devel] HDF5Array failure on windows

2020-04-06 Thread Kasper Daniel Hansen
We currently (and for a while) have had various errors in bsseq that seems to have come and go. We now have a failure on Windows which is related to HDF5. I see that HDF5Array also fails on Windows, which makes me believe the error could be upstream. There is also a warning about hep page links

Re: [Bioc-devel] Updates to SingleCellExperiment class structure - no action required

2020-02-16 Thread Kasper Daniel Hansen
It sounds to me that the class definition / structure is updated. You should also have an associated updateObject() method to deal with this; that's tradition. Of course, you may already have addressed this and just not written about it in your email. On Sun, Feb 16, 2020 at 3:51 AM Aaron Lun <

Re: [Bioc-devel] How to use RData files in Bioconductor data and software packages

2020-01-14 Thread Kasper Daniel Hansen
Tobias, When you use the data() command on the data package, you need to do library(dummyData) first (and you therefore need to Suggest: dummyData) Here is an example from minfi/minfiData if (require(minfiData)) { dat <- preprocessIllumina(RGsetEx, bg.correct=FALSE, normalize="controls") }

Re: [Bioc-devel] proposal for additional seqlevelsStyle

2019-12-13 Thread Kasper Daniel Hansen
If the chromosome name depends on the assembly, that makes GenomeInfoDb even more useful and necessary. Provided it is supported of course. On Fri, Dec 13, 2019 at 11:45 AM Vincent Carey wrote: > I tried an inline png but I think it was rejected by bioc-devel. Here's > another try. > > On

Re: [Bioc-devel] -fopenmp switch issue on mac

2019-12-04 Thread Kasper Daniel Hansen
Are you using the clang 6 (R 3.5) / 7 (R 3.6) / 8 (R 4.0) which Simon supplies? This is for clang 6 with full path /usr/local/clang6/bin/clang --version clang version 6.0.0 (tags/RELEASE_600/final) Target: x86_64-apple-darwin18.7.0 Thread model: posix InstalledDir: /usr/local/clang6/bin On Wed,

Re: [Bioc-devel] CRAN packages depending on broken Bioc 3.11 packages may be archived

2019-11-27 Thread Kasper Daniel Hansen
Or remove the graph dependency On Tue, Nov 26, 2019 at 3:16 PM James W. MacDonald wrote: > Hi Lori, > > Let me know if you don't hear from Benilton. It shouldn't be much work to > fix oligo - we could convert to using Gviz for the graphic that is > currently using GenomeGraphs. I can make the

Re: [Bioc-devel] How to move package from workflow to software

2019-11-15 Thread Kasper Daniel Hansen
Without commenting on whether the move is appropriate, it should trigger a new review process. Best, Kasper On Fri, Nov 15, 2019 at 9:31 AM Julien Wollbrett wrote: > Hello Nitesh, > > Thank you for your answer. > > I added some functionalities to the BgeeCall package that are not > compatible

Re: [Bioc-devel] Opinions on a meta-annotation package

2019-10-24 Thread Kasper Daniel Hansen
>From your description it very much sounds like creating a new package is the way to go. On Thu, Oct 24, 2019 at 3:03 PM Pages, Herve wrote: > Hi Panagiotis, > > Avoiding code repetition is always a good idea. An alternative to the > creation of a 3rd package would be to have one of the 2

Re: [Bioc-devel] check fails on arch 'i386'

2019-10-17 Thread Kasper Daniel Hansen
I see these packages may have something to do with mini. There is a a (very old, nothing new) issue with the annotation objects in minfi where loading them uses a _lot_ of ram. For a long time I couldn't understand it, but now I get it. I should really fix this, and probably _very_soon. It won't

Re: [Bioc-devel] new package for accessing some chemical and biological databases

2019-09-13 Thread Kasper Daniel Hansen
We used to have (? or at least discussed the possibility of) occasional extensive checking so we could have tests long_tests (names made up). On Fri, Sep 13, 2019 at 9:50 AM Martin Morgan wrote: > Putting bioc-devel back in the loop. > > I think that the straight-forward answer to your

Re: [Bioc-devel] RGL warning

2019-09-11 Thread Kasper Daniel Hansen
GLX is a module for X11. You need that module installed in whatever X11 installation you have. My understanding (which is weak) is that GLX is a standard module so it is weird it is missing. On Tue, Sep 10, 2019 at 12:39 PM Zhezhen Wang wrote: > Hi > > I am having the following warnings and not

Re: [Bioc-devel] Failing to build vignette because of problems with ImageMagick

2019-09-11 Thread Kasper Daniel Hansen
r the info! So from my understanding we dont use any > >> trimming or editing function from ImageMagick directly. I think this is > >> rather knitr based since we just include png files in the vignette. > >> > >> I guess it was an hickup since now the error is gone over

Re: [Bioc-devel] Failing to build vignette because of problems with ImageMagick

2019-09-09 Thread Kasper Daniel Hansen
You don't declare any systems requirements for ImageMagick (doing so will probably not solve your problem, but you really should). Alternatively you could look into using the tools provided by the magick package, which wraps ImageMagick. But it looks like you're editing PNG files for your

Re: [Bioc-devel] Import BSgenome class without attaching BiocGenerics (and others)?

2019-09-06 Thread Kasper Daniel Hansen
There are importMethodsFrom(PACKAGE, NAME_OF_METHODS) importClassesFrom(PACKAGE, NAME_OF_METHODS) to help with selective importing S4 methods/classes. See section 1.5.6 of WRE. On Fri, Sep 6, 2019 at 9:24 AM Michael Lawrence via Bioc-devel < bioc-devel@r-project.org> wrote: > It sounds like

Re: [Bioc-devel] Fwd: Re: Build error due to TensorFlow installation

2019-09-05 Thread Kasper Daniel Hansen
It might be true that the code is much easier on your end, but it comes with the complexity of using TensorFlow, which is a huge dependency. And that library can be easy or impossible to install depending on your box. I am not saying people shouldn't use TensorFlow, but I am saying it brings a

Re: [Bioc-devel] 'tokay1' error but passed other OS

2019-09-04 Thread Kasper Daniel Hansen
hat would be great. Please tag me when you do > so I can keep track of it also @lshep on GitHub > Cheers > > Get Outlook for Android <https://aka.ms/ghei36> > > -- > *From:* Venu Thatikonda > *Sent:* Wednesday, September 4, 2019 3:11:56

Re: [Bioc-devel] 'tokay1' error but passed other OS

2019-09-04 Thread Kasper Daniel Hansen
il the issue is resolved. > > > Lori Shepherd > > Bioconductor Core Team > > Roswell Park Cancer Institute > > Department of Biostatistics & Bioinformatics > > Elm & Carlton Streets > > Buffalo, New York 14263 > -- >

Re: [Bioc-devel] 'tokay1' error but passed other OS

2019-09-04 Thread Kasper Daniel Hansen
Well, yes, its the status on the build system which matters. In your case it looks like you're doing BigWig parsing. Unfortunately, some of the tools we have for reading BigWig (tracklayer) does not work reliably on Windows - sometimes it works, sometimes it doesn't. My _guess_ is that you're

Re: [Rd] possible bug in R's configure check for C++11 features

2019-09-04 Thread Kasper Daniel Hansen
CXX11=g++ so it is doing what you asked it to. Since the settings are > inherited upwards, this implies that you are setting both CXX14 and CXX17 > to g++. So I’m not quite sure I understand your concern. > > Cheers, > Simon > > > > > On Sep 3, 2019, at 9:02 PM, Kasper Dani

[Rd] possible bug in R's configure check for C++11 features

2019-09-03 Thread Kasper Daniel Hansen
I am trying to compile R under a new setup, and frankly, I have had a lot of problems, but I think the stuff below points to a possible bug in R's (custom) configure checks for C++11/14/17, but not for C++98. This is a report about R from the R-3-6 branch, with a svn checkout from today, revision

Re: [Bioc-devel] Resetting to previous commits

2019-08-14 Thread Kasper Daniel Hansen
indeed a breaking change, and I thus > believed > > that it would have been counted as a bug-fix for the release branch. > > > > All the best, > > Erik > > > > On 14 Aug 2019, at 17:15, Kasper Daniel Hansen < > > kasperdanielhan...@gmail.com<mail

Re: [Bioc-devel] Resetting to previous commits

2019-08-14 Thread Kasper Daniel Hansen
You cannot revert. By pushing those updates, you released a version into the wild. It is true that as long as it doesn't build on the build servers, it would have been hard to obtain for anyone, but it is still possible through git. You need to update the version. Best, Kasper On Wed, Aug 14,

Re: [Bioc-devel] Resetting to previous commits

2019-08-14 Thread Kasper Daniel Hansen
to "copy" a snapshot of CRAN into release. Best, Kasper On Wed, Aug 14, 2019 at 11:13 AM Kasper Daniel Hansen < kasperdanielhan...@gmail.com> wrote: > You cannot revert. By pushing those updates, you released a version into > the wild. It is true that as long as it does

Re: [Rd] gfortran 9 quantreg bug

2019-08-06 Thread Kasper Daniel Hansen
Dirk, Thanks for the blog post on this, and the pointers in this email. I have a question: it seems to me that you end up using a different compiler for the package (quantreg) than was used to build R itself. As I understand ABI changes, this is considered unsupported (ok, that depends on what

Re: [Bioc-devel] tokay1 C++14 compiler

2019-07-30 Thread Kasper Daniel Hansen
gt; BTW: Recent updates from stan community suggest that the new rstan version > 2.20 will reduce the model compilation time from ~35sec to ~7sec, which > makes the decision of sticking with the run-time compilation setup not so > bad. > On 29.07.19 8:37 nachm., Kasper Daniel Hansen wrote

Re: [Bioc-devel] tokay1 C++14 compiler

2019-07-29 Thread Kasper Daniel Hansen
upported on Windows' (thereby losing 1/2 your > user base). > > > > The expert on this (Hervé) will not be back for several days. > > > > Martin > > > > On 7/25/19, 6:33 PM, "Bioc-devel on behalf of Kasper Daniel Hansen" < > bioc-devel-boun..

Re: [Bioc-devel] tokay1 C++14 compiler

2019-07-25 Thread Kasper Daniel Hansen
What's weird here is that you supposedly depend on rstan which also requires a C++14. How can the build system have that package installed without having a compliant compiler? On Thu, Jul 25, 2019 at 12:05 PM Simo Kitanovski wrote: > Hello Bioconductor community, > > I maintain the

Re: [Bioc-devel] use -Wa, -mbig-obj -O3 when building package under Windows

2019-05-24 Thread Kasper Daniel Hansen
[ This is a crazy bug where you have to increase optimization. ] I some to add here. I am involved in at least two packages which requires overriding the optimization switch to downgrade optimization. You have an additional problem where you need to add additional flags. The solution in

Re: [Bioc-devel] loading database package changes random number

2019-05-23 Thread Kasper Daniel Hansen
to the user, as a dependency-of-a-dependency-of-an > annotation package. > > > > I guess an acceptable solution would be for DelayedArray to remember and > restore the random number seed before creating a BiocParallel param, with > an edge case being that .Random.seed is NULL in

Re: [Bioc-devel] loading database package changes random number

2019-05-22 Thread Kasper Daniel Hansen
Why don't you let this be under the user's control and not do this at all. People should know that reproducibility of random numbers requires setting the seed, but that is best done by the user and not a package author. On Wed, May 22, 2019 at 9:30 AM Steffi Grote wrote: > Hi all, > > I tried

Re: [Bioc-devel] InteractionSet for structural variants

2019-05-21 Thread Kasper Daniel Hansen
ing pipeline in your new coordinate system. A lot more effort, but I'd > have at least some confidence in the results. > > On Wed, May 22, 2019 at 12:07 AM Kasper Daniel Hansen < > kasperdanielhan...@gmail.com> wrote: > >> I know little about SV and the associated softwa

Re: [Bioc-devel] InteractionSet for structural variants

2019-05-21 Thread Kasper Daniel Hansen
I know little about SV and the associated software, but it is clear to me that we will see a lot of "personal" genomes in the future and having the ability to move between different reference genomes (coordinate systems) will be something I think we should think about having good/great support

Re: [Bioc-devel] SummarizedExperiments not equal after serialisation

2019-05-16 Thread Kasper Daniel Hansen
Interesting detective work. This is nasty. Best, Kasper On Thu, May 16, 2019 at 2:19 AM Pages, Herve wrote: > Let's try to go to the bottom of this. But let's leave > SummarizedExperiment objects out of the picture for now and focus on what > happens with a very simple reference object. > >

Re: [Bioc-devel] When to use generics?

2019-05-01 Thread Kasper Daniel Hansen
This is a common situation. Most packages might have a need for something that looks like a generic, but it really only has a usage inside your own package(s). In many ways, the pseudo-generic you define using if() statements is often easier to work with, debugging wise and code-reading wise.

Re: [Bioc-devel] package RBGL requires CRAN dependency on devel branch

2019-03-27 Thread Kasper Daniel Hansen
On Wed, Mar 27, 2019 at 1:06 AM Aaron Lun < infinite.monkeys.with.keyboa...@gmail.com> wrote: > > well, we can't fix this in old branches of Bioc. > > Sure, but one could say that about breaking changes to any CRAN package. > Nothing particularly special about BH on that point. > Indeed, and

Re: [Bioc-devel] package RBGL requires CRAN dependency on devel branch

2019-03-25 Thread Kasper Daniel Hansen
Mon, Mar 25, 2019 at 10:57 AM Kasper Daniel Hansen < > kasperdanielhan...@gmail.com> wrote: > >> There are no issues with depending on CRAN packages. >> >> But I would advise caution. On one hand it is great that boost gets >> updated regularly. On the o

Re: [Bioc-devel] package RBGL requires CRAN dependency on devel branch

2019-03-25 Thread Kasper Daniel Hansen
There are no issues with depending on CRAN packages. But I would advise caution. On one hand it is great that boost gets updated regularly. On the other hand, it could lead to incompatibilities with RBGL and then you have to update that package rapidly. Also - and this is something we could

Re: [Bioc-devel] Call for collaborators/advice

2019-03-22 Thread Kasper Daniel Hansen
Why is this not "just" a function which transforms one GI into another GI? Thats what it seems to me. On Fri, Mar 22, 2019 at 12:31 PM Luke Klein wrote: > I am writing a package that will extend the GenomicInteractions class. I > am a statistician, so I may not know best practices when it

Re: [Bioc-devel] set.seed and BiocParallel

2019-03-12 Thread Kasper Daniel Hansen
But why do you want the same seed for the different features? That is not the right way to use stochastic methods. Best, Kasper On Tue, Mar 12, 2019 at 5:20 PM Bemis, Kylie wrote: > Hi all, > > I remember similar questions coming up before, but couldn’t track any down > that directly pertain

Re: [Bioc-devel] How critical is package style for bioconductor?

2019-02-26 Thread Kasper Daniel Hansen
lintr can be configured to use camelCase. On Tue, Feb 26, 2019 at 11:12 AM Turaga, Nitesh < nitesh.tur...@roswellpark.org> wrote: > As long as it’s consistent, you can use another style. Consistency helps > reviewers read the code easily. > > Most people generally use “camelCase" or “snake_case"

Re: [Bioc-devel] Error package build

2019-02-25 Thread Kasper Daniel Hansen
According to the GitHub repos, the current maintainer is Valentin Dinu Which is not the email you have sent this message from. Do you have multiple email addresses and this is causing this? On Mon, Feb 25, 2019 at 8:44 PM margaret linan wrote: > Hi - > > I have submitted my package PoTRA

Re: [Rd] pmax and long vector

2019-01-21 Thread Kasper Daniel Hansen
, 2019 at 3:09 PM Gabriel Becker wrote: > Kasper, > > If you're not interested or dont have time to create said patch yourself > let me know and i can do it. > > Best, > ~G > > On Mon, Jan 21, 2019, 11:36 AM Duncan Murdoch wrote: > >> On 21/01/2019 12:35 p.m., Ka

[Rd] pmax and long vector

2019-01-21 Thread Kasper Daniel Hansen
I see that base::pmax() does not support long vectors. Is R-devel interested in reports like this; ie. is there a goal of full support for long vectors in "basic" functions, something I at least would greatly appreciate? MRE: > pmax(rep(1L, 3*10^9), 0) Error in pmax(rep(1L, 3 * 10^9), 0) :

Re: [Bioc-devel] Using SerialParam() as the registered back-end for all platforms

2019-01-07 Thread Kasper Daniel Hansen
To add to Henrik's comments, it is also worthwhile to recognize that mclapply() does not deliver statistically sound random numbers even within the apply "loop" unless you use RNGkind("L'Ecuyer-CMRG") which is not set as default. This is because mclapply will initialize random streams with

Re: [Bioc-devel] Questions about some checks in the latest BiocCheck

2018-11-29 Thread Kasper Daniel Hansen
First, the issue with using class() is that the return is a vector possibly with >1 element. This _does_ happen. This is why is() etc is much better,. Second, why do you need case insensitivity when testing for class. That makes NO sense to me. You should be testing for a specific (collection)

Re: [Bioc-devel] Error : object 'lengths' is not exported by 'namespace:BiocGenerics'

2018-11-14 Thread Kasper Daniel Hansen
It seems the short answer is: yes, base::lengths() works and should be fast If it either doesn't work or is slow, you should report it. On Tue, Nov 13, 2018 at 3:33 PM Pages, Herve wrote: > I'm going to try to provide some details, at the risk to confuse you even > more. > > > In BioC 3.8 / R

Re: [Bioc-devel] A little confuse about how to update accepted package

2018-11-10 Thread Kasper Daniel Hansen
Wenhao, The night before the release your package was at 0.99.x in the devel branch At release, two things happened: the package was added to the release branch with version 1.0.0 the package in the devel branch was updated to version 1.1.0 So basically version 0.99.19 == 1.0.0 == 1.1.0.

Re: [Bioc-devel] About the reproducibility of BioC server environment

2018-11-06 Thread Kasper Daniel Hansen
In my experience, if I aggressively update my development installation of Bioconductor (including occasional updates of R-devel when appropriate) I can usually replicate most-all failures on the build server which has the same OS as my local machine. It does require aggressive updating though.

Re: [Bioc-devel] macOS Mojave and R 3.6: issue with building some packages from source

2018-11-04 Thread Kasper Daniel Hansen
You seem to install the Beta version of the Command Line tools. That's not right. Look at the top of apple.com/developer/downloads (First try with xcode-select --install on the command line, which I could not get to work). Following installation of these tools you need to run the following

Re: [Bioc-devel] GenomicFeatures

2018-11-02 Thread Kasper Daniel Hansen
her CRAN nor Bioconductor. > > Though of course we'd like GenomicFeatures to be available in 3.8 > > Martin > > On 11/2/18, 1:05 PM, "Bioc-devel on behalf of Kasper Daniel Hansen" < > bioc-devel-boun...@r-project.org on behalf of kasperdanielhan...@gmail.com> > wr

[Bioc-devel] GenomicFeatures

2018-11-02 Thread Kasper Daniel Hansen
Seem broken in release. And btw., because it is broken, when I run BiocManager::valid() it doesn't report GenomicFeatures as out of date, despite the fact that I have version 1.33.2 installed, and there are newer versions in the 1.33.x hierarchy and the "right" (but broken) version in Bioc 3.8

Re: [Bioc-devel] Ask help to install gs tokay2 build/check

2018-11-02 Thread Kasper Daniel Hansen
Perhaps explain why On Fri, Nov 2, 2018 at 8:19 AM Jianhong Ou, Ph.D. wrote: > Hi, > > Is it possible to install ghostscript in tokay2 for development package > build? > > Yours Sincerely, > > Jianhong Ou > > Email: jianhong...@duke.edu > Bioinformatician II >

Re: [Bioc-devel] Dependency to PEER

2018-09-25 Thread Kasper Daniel Hansen
It is a major weakness of PEER that it has not been submitted to a standard repository. Of course, that is up to the PEER developers, but the community should put pressure on them. On Tue, Sep 25, 2018 at 12:40 PM Christian Mertes wrote: > Hi all, > > Im the developer of OUTRIDER. Our package

Re: [Bioc-devel] package c++ optimization flag

2018-09-19 Thread Kasper Daniel Hansen
This should be done extremely sparingly, but it is sometimes necessary if you know that higher optimization breaks the code. But that is the only use case. Otherwise you should trust the setting of the user who compiles R, ie. you should for example never force a higher optimization level IMO. We

Re: [Bioc-devel] Block bootstrap for GenomicRanges

2018-08-14 Thread Kasper Daniel Hansen
I agree this is super important. I think there may be multiple ways of thinking about a decent bootstrapping or permutation of ranges, in genomics. I am quite interested in the topic. I think it might belong in a new package. I would be interesting in extending the conversation and have a couple

Re: [Bioc-devel] updating maintainer

2018-06-27 Thread Kasper Daniel Hansen
You update the maintainer field in devel. I don't know what to do about release. I don't understand the question about github. There is no "github email", there is a github username. And that username does not have to be the same as a maintainer email, because they really have nothing to do

Re: [Bioc-devel] Including a submodule in the main package repository

2018-06-14 Thread Kasper Daniel Hansen
My opinion: maintain two repos and pull/push between them. You would want the ability to have them out of sync anyway. Best, Kasper On Thu, Jun 14, 2018 at 1:16 AM, Thomas Sherman wrote: > Hi all, > > I'm a developer for the CoGAPS package (https://github.com/FertigLab/ > CoGAPS). I'm looking

Re: [Bioc-devel] Package size note

2018-05-15 Thread Kasper Daniel Hansen
Kenneth, You should _always_ run R CMD check (and R CMD BiocCheck) on the tarball generated by R CMD build. The build step essentially cleans the code. If your vignette takes a bit to generate (the vignette gets generated with R CMD build), just do R CMD build --no-build-vignettes Best, Kasper

  1   2   3   4   5   >