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

Notebook improvements #292

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
4 changes: 2 additions & 2 deletions alphabase/constants/aa.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ def calc_sequence_masses_for_same_len_seqs(sequence_array: np.ndarray) -> np.nda

def calc_AA_masses_for_var_len_seqs(sequence_array: np.ndarray) -> np.ndarray:
"""
We recommend to use `calc_AA_masses_for_same_len_seqs` as it is much faster.
We recommend to use `calc_AA_masses_for_same_len_seqs` as it is much faster. # TODO it's the same

Parameters
----------
Expand All @@ -175,7 +175,7 @@ def calc_AA_masses_for_var_len_seqs(sequence_array: np.ndarray) -> np.ndarray:
Returns
-------
np.ndarray
1D array of masses, zero values are padded to fill the max length.
1D array of masses, values of 1e8 are used to fill the max length.
"""
return AA_ASCII_MASS[np.array(sequence_array).view(np.int32)].reshape(
len(sequence_array), -1
Expand Down
Loading
Loading