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

Fix installation #87

Merged

Conversation

william-hutchison
Copy link
Contributor

A few bug fixes:

  • Fix the Azimuth package remote address (this bug means HPCell cannot currently install).
  • Fix incorrectly attributed function imports.
  • Fix a typo in the README.

@william-hutchison
Copy link
Contributor Author

A few more bug fixes to address all R CMD Errors:

  • Add a missing remote.
  • Add multiple packages imported from in NAMESPACE to DESCRIPTION imports.
  • Updated an example with :: notation.
  • Removed inheritance of examples for test_differential_abundance from tidybulk as they were failing. I think they could be fixed in tidybulk by declaring all imports with :: notation in relevant examples.

GIthub actions are showing failure because of R CMD Warnings. Fixing these is probably beyond the scope of this PR. I will also attempt to solve the bug I encountered in SCTransform in a separate PR.

@stemangiola stemangiola merged commit 81419ca into MangiolaLaboratory:master Dec 5, 2024
0 of 3 checks passed
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.

2 participants