Skip to content

Multi-image frontend + minor updates t owebsite#66

Merged
amirDahari1 merged 29 commits intomainfrom
development
Jul 30, 2025
Merged

Multi-image frontend + minor updates t owebsite#66
amirDahari1 merged 29 commits intomainfrom
development

Conversation

@rmdocherty
Copy link
Collaborator

No description provided.

@amirDahari1 amirDahari1 requested a review from Copilot July 30, 2025 13:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds multi-image support to the ImageRep frontend, allowing users to upload multiple images for batch analysis, along with minor updates to the website including typo fixes and code formatting improvements.

  • Multi-image frontend functionality with batch processing capability
  • Website updates including typo corrections and formatting improvements
  • Server-side modifications to handle multiple file uploads with async support

Reviewed Changes

Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
requirements dev.txt Adds Flask[async] dependency for async server support
representativity/server.py Updates server endpoints to handle multiple files with async functions
representativity/core.py Code formatting improvements and minor refactoring
frontend/ files Extensive frontend updates for multi-image support, UI improvements, and code formatting
README.md Updates paper reference links and formatting improvements
CITATION.cff Updates citation information with new paper reference
Comments suppressed due to low confidence (2)

requirements dev.txt:8

  • Flask version 3.1.0 does not exist. The latest Flask 3.x versions are 3.0.x series. Consider using Flask[async]==3.0.3 or check the correct version number.
Flask[async]==3.1.0

representativity/server.py:147

  • [nitpick] The parameter name 'image_stack' is misleading as it's now being used to indicate whether there are multiple images rather than specifically a stack. Consider renaming to 'multiple_images' or 'batch_mode' for clarity.
        arrs, selected_conf, selected_err, model_error=True, image_stack=is_stack

@amirDahari1 amirDahari1 self-requested a review July 30, 2025 13:47
@amirDahari1 amirDahari1 merged commit 2c51ffb into main Jul 30, 2025
1 check 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.

3 participants