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

version 0.13.1 with fixes for drqng CRAN checks #151

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

s3alfisc
Copy link
Owner

No description provided.

@s3alfisc s3alfisc linked an issue May 13, 2024 that may be closed by this pull request

set.seed(123123)
dqrng::dqRNGkind("Xoroshiro128+")
dqrng::dqset.seed(123123)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the above lines are necessary. I had included them in my reprex in order to be sure that the difference in confidence intervals is larger than 0.005. However, I think it is in principle a good idea to run this kind of tests w/o fixing the seed or RNG.

@zeileis
Copy link

zeileis commented Jul 30, 2024

@s3alfisc and @rstub thanks for all your work on the fwildclusterboot and dqrng updates.

Are there any news regarding the CRAN update? I'm also asking because of: cran-task-views/Econometrics#25

@rstub
Copy link

rstub commented Aug 5, 2024

AFAIK one problem is that there has been a lot of development here on GH that has not been published to CRAN yet. So fixing only this thing for CRAN is a bit complex, but not impossible.
@s3alfisc Please let me know if I can be of any help here.

@zeileis
Copy link

zeileis commented Aug 5, 2024

Alexander @s3alfisc if it is hard to release the current development version, then please consider backporting the essential fixes to the current CRAN version and release that. If I can help with anything, let me know.

@rstub
Copy link

rstub commented Aug 5, 2024

In rstub/fwildclusterboot@dqrng-fix I have put in the minimal fix that should help getting this back unto CRAN. AFAIK we would need a branch based on the v0.13.0 tag to make a PR from this.

@zeileis
Copy link

zeileis commented Aug 5, 2024

Great, sounds like a good plan!

@s3alfisc
Copy link
Owner Author

s3alfisc commented Aug 5, 2024

Hi both, thanks for nudging me to get the package back on CRAN! I tried at the end of my 2 week grace period but failed (which was frustrating) and then I got some not-so encouraging feedback on top. I have now addressed all comments & the changes suggested by @rstub after reverting to 0.13.0 in this PR #153 and will now run checks on rhub and win devel. Will then release if both pass, hopefully today =)

@rstub
Copy link

rstub commented Aug 5, 2024

You will have to wait until 2024-08-16, since CRAN is on holiday right now.

@s3alfisc
Copy link
Owner Author

s3alfisc commented Aug 5, 2024

Ah, thanks! 😅😀 so still some days to fix all these broken URL's that have been found in the checks 😀

@zeileis
Copy link

zeileis commented Sep 23, 2024

Alexander @s3alfisc, another nudge... 👋

@s3alfisc
Copy link
Owner Author

s3alfisc commented Sep 23, 2024

Just send out a new round of tests to win develop. Package is already back on r-Universe (since yesterday). Thanks for the reminder @zeileis!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sampling tests to strict
3 participants