Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025

Removes legacy example files from src/examples/ as requested.

Removed files

  • PhosphoScoring.py
  • runPhosphoScoring.py
  • GUI_EXAMPLE_BASE.py
  • GUI_ErrorWidget.py
  • GUI_MS1MapWidget.py
  • GUI_ScanBrowserWidget.py
  • GUI_ScanTableWidget.py
  • GUI_SequenceIonsWidget.py
  • GUI_SpectrumWidget.py
  • GUI_TICWidget.py

Updated files

  • README.md - removed runPhosphoScoring.py documentation entry
Original prompt

remove PhosphoScoring.py and all files with GUI_*.py in the examples folder


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@coderabbitai
Copy link

coderabbitai bot commented Dec 15, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Note

Free review on us!

CodeRabbit is offering free reviews until Wed Dec 17 2025 to showcase some of the refinements we've made.

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: timosachsenberg <5803621+timosachsenberg@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove PhosphoScoring.py and GUI example files Remove PhosphoScoring.py and GUI_*.py files from examples Dec 15, 2025
@timosachsenberg
Copy link
Contributor

@copilot check if these files are anywhere referenced or used

Copy link
Contributor Author

Copilot AI commented Dec 15, 2025

@copilot check if these files are anywhere referenced or used

I checked for references to the removed files:

PhosphoScoring.py is referenced in:

  • src/examples/runPhosphoScoring.py (line 4) - imports PhosphoScorerSimple, PhosphoScorerAScore from it
  • src/examples/README.md (line 23) - documents runPhosphoScoring.py

GUI_*.py files - No external references found. They only imported from each other internally.

The runPhosphoScoring.py script is now broken since it imports from the removed PhosphoScoring.py. Would you like me to also remove runPhosphoScoring.py and update the README.md to remove its documentation entry?

@timosachsenberg
Copy link
Contributor

@copilot remove runPhosphoScoring.py and update readme

Co-authored-by: timosachsenberg <5803621+timosachsenberg@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 15, 2025

@copilot remove runPhosphoScoring.py and update readme

Done in b4f16d4. Removed runPhosphoScoring.py and updated README.md to remove its documentation entry.

Copilot AI changed the title Remove PhosphoScoring.py and GUI_*.py files from examples Remove PhosphoScoring.py, runPhosphoScoring.py and GUI_*.py files from examples Dec 15, 2025
@timosachsenberg timosachsenberg marked this pull request as ready for review December 15, 2025 21:43
@timosachsenberg timosachsenberg merged commit e2df208 into master Dec 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants