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

Error in eval(expr, envir, enclos) : objeto 'Stat' não encontrado #18

Closed
pedrostrusso opened this issue May 22, 2017 · 11 comments
Closed
Labels

Comments

@pedrostrusso
Copy link
Collaborator

Apparently it has to do with the sna package. Looking into it.

@lecardozo
Copy link
Collaborator

I think we already faced this bug before. There's this issue on the ggnetwork repo that was fixed in this PR. Maybe the CRAN version is not synced with the github repo.
To test that we could add ggnetwork as a remote on DESCRIPTION and see what happens.
I don't know how to reproduce this error. Do you?

@pedrostrusso
Copy link
Collaborator Author

No, I haven't faced it in a while actually, Helder had the issue on his PC. I'll check his package versions later to see if it matches mine.

@csbl-usp
Copy link
Owner

csbl-usp commented May 22, 2017 via email

@csbl-usp
Copy link
Owner

csbl-usp commented May 22, 2017 via email

@csbl-usp
Copy link
Owner

csbl-usp commented May 23, 2017 via email

@pedrostrusso
Copy link
Collaborator Author

As @matheuscburger and @lecardozo pointed out, the issue is accounted for and fixed in the ggnetwork GitHub page. However, I downloaded the tar.gz from CRAN and saw that the changes they made to the package weren't updated. So is there a way for us to make a user's pc download the package from the GitHub repo instead of a CRAN mirror when CEMiTool is being installed?

@matheuscburger
Copy link
Collaborator

matheuscburger commented May 24, 2017 via email

@pedrostrusso
Copy link
Collaborator Author

It seems like we fixed the problem. @lecardozo forked the ggnetwork repo and edited the functions to import Stat from ggplot2, and added these imports to our NAMESPACE file.

@pedrostrusso
Copy link
Collaborator Author

pedrostrusso commented May 31, 2017

This error appeared during the Bioconductor build attempts. More details at:
http://bioconductor.org/spb_reports/CEMiTool_buildreport_20170531132739.html

Maybe Bioconductor isn't getting the package from the ggnetwork fork @lecardozo created? Do you guys think that if we import Stat ourselves in the CEMiTool package (instead of doing it in the ggnetwork package) it could work?

@pedrostrusso
Copy link
Collaborator Author

I removed the ggnetwork package dependency. There should be no more issue now.

@briatte
Copy link

briatte commented Feb 12, 2020

Dear all ~ FWIW, ggnetwork has just been (heavily) updated.

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

No branches or pull requests

5 participants