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

This push request add the ligand/receptor data to the package and uses package data within functions #140

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

Conversation

Xiaojieqiu
Copy link
Contributor

Firstly, I added a __init__.py file in the spateo/tools/database (where the ligand-receptor files are stored)

Then I added a MANIFEST.in file, understand the same folder of setup.py, with the following text:
recursive-include spateo/tools/database *

Then the setup.py file included the line of
include_package_data=True,

We use the following code to access the package data:
path = importlib.resources.path("spateo", "data") if path is None else path

Read more here and here, especially the second answer

Code in the cell_communication.py are better formated and missing documentation of args are included

@Xiaojieqiu Xiaojieqiu requested a review from luluZuo August 4, 2022 16:25
@Xiaojieqiu
Copy link
Contributor Author

@luluZuo can we have these files:

ligand_target_matrix and ligand_target_matrix_mouse to the database folder? Is it too big?

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.

1 participant