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 the following metric: true reactant accuracy #4

Merged
merged 16 commits into from
Aug 2, 2023

Conversation

avaucher
Copy link
Member

@avaucher avaucher commented Aug 2, 2023

No description provided.

@cla-bot cla-bot bot added the cla-signed label Aug 2, 2023
from rxn.metrics.utils import get_sequence_multiplier


def test_get_sequence_multiplier() -> None:
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: just moved from the other file

round_trip_accuracy,
top_n_accuracy,
)


def test_get_sequence_multiplier() -> None:
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: moved it to another file

)


def get_sequence_multiplier(ground_truth: Sequence[T], predictions: Sequence[T]) -> int:
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: moved from another file

@@ -161,17 +162,3 @@ def class_diversity(
}
std_dev = {i + 1: float(np.std(classes_for_n[i])) for i in range(multiplier)}
return classdiversity, std_dev


def get_sequence_multiplier(ground_truth: Sequence[T], predictions: Sequence[T]) -> int:
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: moved to another file

@avaucher avaucher marked this pull request as ready for review August 2, 2023 09:23
@avaucher avaucher merged commit a987053 into main Aug 2, 2023
2 checks passed
@avaucher avaucher deleted the true-reactant-metric branch August 2, 2023 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant