Skip to content

Add method and metric descriptions - #810

Merged
scottgigante-immunai merged 3 commits into
openproblems-bio:mainfrom
scottgigante-immunai:docs/metric_descriptions
Feb 28, 2023
Merged

Add method and metric descriptions#810
scottgigante-immunai merged 3 commits into
openproblems-bio:mainfrom
scottgigante-immunai:docs/metric_descriptions

Conversation

@scottgigante-immunai

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Feb 1, 2023

Copy link
Copy Markdown
Contributor

Current build status

@rcannood

Copy link
Copy Markdown
Member

What needs to be done before we can merge this pr? :)

@scottgigante-immunai

Copy link
Copy Markdown
Collaborator Author

Pending methods descriptions from @danielStrobl @giovp @wes-lewis in the supplementary information doc. Once those are in, this can be merged.

@giovp

giovp commented Feb 19, 2023

Copy link
Copy Markdown
Collaborator

@scottgigante-immunai @LuckyMD think all should be done from my side? let me know if anything else is needed.

@scottgigante-immunai

Copy link
Copy Markdown
Collaborator Author

@giovp I'm only missing NMF from you. @danielStrobl and @mxposed still waiting on your method descriptions.

@giovp

giovp commented Feb 27, 2023

Copy link
Copy Markdown
Collaborator

@scottgigante-immunai added vanilla nmf

@scottgigante-immunai
scottgigante-immunai marked this pull request as ready for review February 27, 2023 16:25

@rcannood rcannood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just one minor change requested. LGTM!

Comment thread openproblems/tasks/_cell_cell_communication/_common/metrics/auprc.py Outdated
@scottgigante-immunai
scottgigante-immunai merged commit b3456fd into openproblems-bio:main Feb 28, 2023
@scottgigante-immunai
scottgigante-immunai deleted the docs/metric_descriptions branch February 28, 2023 14:20
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.

3 participants