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

bug: analysis.correlate_experiments() #20

Open
aowen-uwmad opened this issue Aug 6, 2020 · 3 comments
Open

bug: analysis.correlate_experiments() #20

aowen-uwmad opened this issue Aug 6, 2020 · 3 comments
Labels
bug enhancement medium-effort Requires more than a few lines of code to fix

Comments

@aowen-uwmad
Copy link
Contributor

Was unable to use the analysis.correlate_experiments() due to a NameError.
Line 1003 referenced in the traceback is the only occurrence of isin_array in analysis.py module, so either it was defined in a different module and not called correctly, or is not defined at all.
The experiments passed were test-of-concept using the same data set, but with different thresholds for the peak finding. The experiment sessions seem to work fine otherwise.

---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-27-a7140606d294> in <module>
----> 1 analysis.correlate_experiments((pyrimidinetest,pyrimidine))

...\Anaconda\envs\pst\lib\site-packages\pyspectools\spectra\analysis.py in correlate_experiments(experiments, thres_prox, index)
   1001                     experiment.line_lists["Peaks"].frequencies
   1002                 )
-> 1003                 mask = isin_array(base_freqs, comp_freqs, thres_prox)
   1004                 # Convert to boolean mask
   1005                 mask.dtype = bool

NameError: name 'isin_array' is not defined
@laserkelvin
Copy link
Owner

Yup this function is currently broken and needs to be fixed: its functionality is supposed to be for comparing frequency coincidences between experiments, which is particularly useful for figuring out unknown molecules appearing in select reactions or conditions. Do you have an immediate use for this?

@laserkelvin laserkelvin added the bug label Aug 6, 2020
@aowen-uwmad
Copy link
Contributor Author

I do not have an immediate use. I was curious what it outputted to make sure #21 wasn't already addressed.

@laserkelvin laserkelvin added enhancement medium-effort Requires more than a few lines of code to fix labels Aug 7, 2020
@laserkelvin
Copy link
Owner

In that case, I suggest keeping this open: I think the function will be of use, but probably better if someone has an immediate use case work on developing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement medium-effort Requires more than a few lines of code to fix
Projects
None yet
Development

No branches or pull requests

2 participants