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

dynamically choosing proteins from volcano plot #389

Draft
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

boopthesnoot
Copy link
Collaborator

@boopthesnoot boopthesnoot commented Dec 20, 2024

Added functionality to choose which proteins to look up in uniprot and subsequently analyze with an LLM

Copy link
Contributor

@mschwoer mschwoer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that feature already :-)

alphastats/gui/pages/06_LLM.py Outdated Show resolved Hide resolved
@@ -202,14 +202,14 @@ def gather_parameters_and_do_analysis(
raise ValueError(f"Analysis method {analysis_method} not found.")


def gather_uniprot_data(features: list) -> None:
def gather_uniprot_data(features: list[str]) -> None:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List[str] .. we're still at 3.8 :-)

alphastats/gui/pages/06_LLM.py Outdated Show resolved Hide resolved
upregulated_genes = [
key for key in regulated_genes_dict if regulated_genes_dict[key] == "up"
]
downregulated_genes = [
key for key in regulated_genes_dict if regulated_genes_dict[key] == "down"
]

# Create dataframes with checkboxes for selection
upregulated_genes_df = pd.DataFrame(
{"Protein": upregulated_genes, "Selected": [True] * len(upregulated_genes)}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This displays a different type of if than the volcano and the rest of the page
image

check out st.session_state[StateKeys.DATASET]._feature_to_repr_map

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's the consensus? do we use genes or proteins everywhere?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DataSet._create_id_dicts defines the current consensus on the representation

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

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