Closed cosimameyer closed 1 year ago
@cosimameyer This error is really weird. I am not able to reproduce it. Can you double checkt that packageVersion("signnet")
gives 1.0.0?
The version number confusion is my fault. I thought it would be better to release 1.0.0 only after the review process. In hindsight, it would have been smarter to send 1.0.0 to CRAN and after the review push a bugfix release. If the package is accepted, I will submit the update directly to CRAN so that there are no inconsitancies. I think that is a better solution than talking about the dev version in the paper.
I cannot fully explain why but I've got good news: it works now with v1.0.0 and I can reproduce your paper π
(to better understand what happened: I re-ran the short script that I used yesterday - in a new R project today; I didn't use packageVersion()
but the RStudio GUI to check for the package version (probably that was the problem that it was a bit laggy in RStudio's interface and didn't display the correct version number))
In order to get the paper accepted, you would need to slightly change the order of your plans: First submit v1.0.0 to CRAN (because this is the basis of your paper and you give instructions for an installation as if the version was already on CRAN) and then get the paper accepted π
Otherwise you have a paper out there that cannot be replicated in its current version until CRAN approves the new version of your package.
Yeah this makes far more sense than my original plans. I will submit it to CRAN today and let you know when it is accepted there
@cosimameyer submitted and it was already accepted :open_mouth:
That's fantastic news! (A bit like a Christmas miracle) π₯³ I'll check the tests tomorrow and if they all pass, I'm very happy to close this issue βΊοΈ
Great thanks! :santa:
When reproducing the results presented in the paper, I experienced a small hic-up with the package versions. In the paper, you install the CRAN version (0.8.1) but use
graph_from_adjacency_matrix_signed()
in the "Centrality" section (which was, based on your release statements, introduced in 1.0.0 and currently the development version). To replicate the results, you'd need to also introduce the development package version call here.When using version 1.0.0, however, I'm running into (exporting) errors and cannot reproduce the following code snippet from your paper:
It looks like the function was not correctly exported. I'm not 100% sure why this happens. You export the function
graph_from_adjacency_matrix_signed ()
with@export
and it's also listed in NAMESPACE - so everything looks good from this perspective. Within thesignnet
R-project package environment, it also works perfectly after loading the package. But when calling the function withsignnet::
(or evensignnet:::
- just to eliminate all options) in another project, I get the following errors:Calling
ls("package:signnet")
returns the following output wheregraph_from_adjacency_matrix_signed
is not listed:Here is my session info:
(This issue is part of the review: https://github.com/openjournals/joss-reviews/issues/4987)