Skip to content

Integrate --config workflow into mea_pipeline_gui.py + fix fixed_y propagation#135

Draft
mandarmp with Copilot wants to merge 3 commits into
mainfrom
copilot/integrate-ipnanlysis-gui-config
Draft

Integrate --config workflow into mea_pipeline_gui.py + fix fixed_y propagation#135
mandarmp with Copilot wants to merge 3 commits into
mainfrom
copilot/integrate-ipnanlysis-gui-config

Conversation

Copilot AI commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

The GUI had no support for the --config JSON workflow, requiring users to pass all settings as CLI flags. Additionally, generate_reports() was missing fixed_y in its signature, causing a TypeError at runtime.

GUI: CONFIG FILE section

  • New CONFIG FILE section at top of GUI with path entry + Browse button
  • Load Config → populate fields: reads a JSON config and pre-fills output_dir, checkpoint_dir, sorter, docker_image, reference_file, export_to_phy, clean_up, no_curation, fixed_y
  • Save Config ← current fields: serializes current GUI state to a config JSON matching config_loader.py's schema
  • --config <path> injected into the generated command when set; CLI flag overrides are preserved

Bug fix: fixed_y propagation through generate_reports()

generate_reports() accepted no fixed_y param but callers passed fixed_y=fixed_y, causing a crash:

# Before — TypeError at runtime
def generate_reports(self, ..., raster_sort=None):
    self._run_burst_analysis(clean_units, ..., raster_sort=raster_sort)

# After
def generate_reports(self, ..., raster_sort=None, fixed_y=False):
    self._run_burst_analysis(clean_units, ..., raster_sort=raster_sort, fixed_y=fixed_y)

Minor

  • Removed duplicate fixed_y checkbox entry in OPTIONS list
Original prompt

This section details on the original issue you should resolve

<issue_title>Think of a way to integrate Shruti IPNANALYSIS GUI to the config style run.</issue_title>
<issue_description></issue_description>

<agent_instructions>mea_pipeline_gui.py to work with the --config based workflow</agent_instructions>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: mandarmp <33731455+mandarmp@users.noreply.github.com>
Copilot AI changed the title [WIP] Integrate Shruti IPNANALYSIS GUI with config style run Integrate --config workflow into mea_pipeline_gui.py Mar 2, 2026
…om main)

Co-authored-by: mandarmp <33731455+mandarmp@users.noreply.github.com>
Copilot AI changed the title Integrate --config workflow into mea_pipeline_gui.py Integrate --config workflow into mea_pipeline_gui.py + fix fixed_y propagation Mar 3, 2026
@mandarmp mandarmp force-pushed the copilot/integrate-ipnanlysis-gui-config branch from dc5cdb3 to 68c7544 Compare April 28, 2026 23:10
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.

Think of a way to integrate Shruti IPNANALYSIS GUI to the config style run.

2 participants