Proposed Organization and Place for AI and Heuristic Predictions of BioPolymer properties#982
Proposed Organization and Place for AI and Heuristic Predictions of BioPolymer properties#982trishorts wants to merge 9 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #982 +/- ##
=======================================
Coverage 81.09% 81.09%
=======================================
Files 271 271
Lines 39144 39144
Branches 4292 4292
=======================================
Hits 31744 31744
Misses 6650 6650
Partials 750 750 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR establishes the organizational structure for a new "Predictions" project in mzLib, intended to house AI and heuristic prediction functionality for biopolymer properties. The PR adds placeholder classes for various prediction types (cross-section, detectability, elution/retention, and fragmentation) along with infrastructure classes for web clients and interfaces.
- Added new Predictions project targeting .NET 8.0 to the solution
- Created stub classes for different biopolymer property prediction types
- Updated solution file with build configurations for the new project
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| mzLib/mzLib.sln | Added Predictions project and "Any CPU" build configurations; updated Visual Studio version |
| mzLib/Predictions/Predictions.csproj | New project file targeting .NET 8.0 with nullable reference types enabled |
| mzLib/Predictions/BioPolymerCrossSection.cs | Placeholder class for cross-section predictions |
| mzLib/Predictions/BioPolymerDectectability.cs | Placeholder class for detectability predictions (contains spelling error) |
| mzLib/Predictions/BioPolymerElutionRetention.cs | Placeholder class for elution/retention time predictions |
| mzLib/Predictions/BioPolymerFragmentation.cs | Placeholder class for fragmentation predictions |
| mzLib/Predictions/Interfaces.cs | Placeholder class for interfaces (ambiguous naming) |
| mzLib/Predictions/WebClients.cs | Placeholder class for web client functionality (generic naming) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| namespace Predictions | ||
| { | ||
| internal class BioPolymerDectectability |
There was a problem hiding this comment.
The class name "BioPolymerDectectability" has a spelling error. It should be "BioPolymerDetectability" (with two 't's in "Detectability").
| internal class BioPolymerDectectability | |
| internal class BioPolymerDetectability |
No description provided.