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

Add example config file in the help documentation #51

Open
lizhiweiyy opened this issue Feb 20, 2023 · 1 comment
Open

Add example config file in the help documentation #51

lizhiweiyy opened this issue Feb 20, 2023 · 1 comment

Comments

@lizhiweiyy
Copy link

Is your feature request related to a problem? Please describe.
All the examples in the documentation have used a config file with the following code:

import conf #This is the local config file named conf.py
odir = conf.SUPPLEMENTS_DIR
kinase_dict[kinase] = conf.KINASE_MAP.loc[kinase,'Preferred Name']

This indicates we need to create a custom file ‘conf.py’ in the working directory, but there isn't any example file.
There are two attributes in the file:
SUPPLEMENTS_DIR, which is a string variable indicating a file path,
and KINASE_MAP, which is a kinase dictionary object, however, it's not clear how to define the variable ‘KINASE_MAP’ in the config file.

Describe the solution you'd like
Add an example file conf.py for download in the help documentation

@srcrowl
Copy link
Contributor

srcrowl commented Feb 20, 2023

Thanks for bringing this to our attention, sorry for the missing information. The conf.py file is very simple, and is really just used to indicate the directory containing the supplementary data, and then load in the kinase map file. At a minimum, all the conf.py file needs to contain is the following:

import pandas as pd

#where supplementary data was downloaded to
SUPPLEMENTS_DIR = './'

#load kinase map from supplementary data
KINASE_MAP =  pd.read_csv(SUPPLEMENTS_DIR+'SupplementaryData/Map/globalKinaseMap.csv', index_col = 0)`

Alternatively, you could avoid using an external file and just include this code in the same python script.

We will update the documentation accordingly, and will close this issue once the issue is clarified.

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

No branches or pull requests

2 participants