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

Johnny Deconv 1st Step #791

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Conversation

nbollis
Copy link
Contributor

@nbollis nbollis commented Aug 2, 2024

Added the boilerplate for John to go in and add his dlll

Copy link

codecov bot commented Aug 2, 2024

Codecov Report

Attention: Patch coverage is 9.43396% with 48 lines in your changes missing coverage. Please review.

Project coverage is 75.34%. Comparing base (a78eea2) to head (7c5132e).

Files Patch % Lines
...ometry/Deconvolution/Algorithms/IsoDecAlgorithm.cs 0.00% 41 Missing ⚠️
...lution/Parameters/IsoDecDeconvolutionParameters.cs 0.00% 5 Missing ⚠️
...Lib/MassSpectrometry/Deconvolution/Deconvoluter.cs 66.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #791      +/-   ##
==========================================
- Coverage   75.46%   75.34%   -0.12%     
==========================================
  Files         201      203       +2     
  Lines       30814    30859      +45     
  Branches     3106     3111       +5     
==========================================
- Hits        23254    23251       -3     
- Misses       7035     7082      +47     
- Partials      525      526       +1     
Files Coverage Δ
...lution/Algorithms/ClassicDeconvolutionAlgorithm.cs 96.89% <100.00%> (-0.04%) ⬇️
...Lib/MassSpectrometry/Deconvolution/Deconvoluter.cs 90.47% <66.66%> (-9.53%) ⬇️
...lution/Parameters/IsoDecDeconvolutionParameters.cs 0.00% <0.00%> (ø)
...ometry/Deconvolution/Algorithms/IsoDecAlgorithm.cs 0.00% <0.00%> (ø)

@trishorts
Copy link
Contributor

maybe we can add something for each new decon method about which of the two camps it falls into.

#694

[StructLayout(LayoutKind.Sequential, Pack =1)]
public struct MatchedPeak
{
public float mz;
Copy link
Contributor

Choose a reason for hiding this comment

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

would be good to use our standard variable names for these features.

foreach(MatchedPeak peak in matchedpeaks)
{
List<(double,double)> peaks = new List<(double,double)> ();
List<double> listofratios = new List<double>();
Copy link
Contributor

Choose a reason for hiding this comment

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

better variable name

@trishorts
Copy link
Contributor

What mass range and resolution will this method apply to?

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.

2 participants