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

New Tool: FRiX Calculator and restructure main GUI #155

Merged
merged 5 commits into from
Jun 22, 2024
Merged

New Tool: FRiX Calculator and restructure main GUI #155

merged 5 commits into from
Jun 22, 2024

Conversation

owlang
Copy link
Collaborator

@owlang owlang commented May 31, 2024

Restructure main GUI to reduce code repetition and add new FRiX Calculator

owlang added 5 commits August 4, 2023 14:10
#138

Most tool group tab JPanels switched to BoxLayout with center stacking of function-initiated JPanels (initializeToolPanel). These were nested in a JScrollPane with vertical scrolling enabled as needed and horizontal scrolling disabled.

Each intializeToolPanel used the GridBagLayout with 5px padding between components.
Move the ExcelAdapter JTable structure used by the scale-matrix tool to its own class within the objects package. This essentially enables pasting with Ctrl-V to the PastableeTable object (copy functions already enabled).
- update TagPileup to throw ScriptManager OptionExceptions for invalid PileupParameter values instead of "This should not print" print statements
- also throw OptionException during CLI validation of input options
- add javadocs to `getCLIcommand()` method
- handle (catch and throw) exceptions for window_interface classes
charts.Histogram
-  include new histogram type formatted for FRiX tool
cli.Peak_Analysis.FRiXCalculatorCLI
- write CLI with Javadoc and Picocli decorators, validation methods, and CLI command building methods based on TagPileup (with unused elements stripped)
main.ScriptManagerGUI
- add FRiX Score Calculator to Peak_Analysis tab
objects.ToolDescriptions
- add FRiX Calculator description
scripts.Peak_Analysis.FRiXCalculator
- write FRiXCalculator script based on TagPileup and leveraging PileupExtract helper script
util.BAMUtilities
- modify BAMUtilities helper script for getting StandardizationRatio such that TagPileup behavior is unchanged but separates out statistics needed by FRiXCalculator
window_interface.Peak_Analysis.FRiXCalculatorOutput/Window
- build GUI interface for user selection of options and to display output
@owlang owlang requested a review from WilliamKMLai May 31, 2024 15:23
@owlang owlang merged commit ddf3622 into dev Jun 22, 2024
6 checks passed
@owlang owlang deleted the frix-score branch June 22, 2024 23:50
@owlang
Copy link
Collaborator Author

owlang commented Aug 2, 2024

#6 #138

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.

1 participant