Skip to content

Added Data Import Overview with Batch Processing Status #23

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Aug 11, 2025
Merged

Added Data Import Overview with Batch Processing Status #23

merged 22 commits into from
Aug 11, 2025

Conversation

@ZyuT0h
Copy link
Contributor

@ZyuT0h ZyuT0h commented Aug 1, 2025 •
edited
Loading

What is done in this PR?

BatchCard component: to put batch information.

  • 1 image that represent the batch
  • Number of Artefacts/ Total of Artefacts Processed [To be updated]
  • Timestamp (Today/Yesterday/Date format [2 AUG 2025] and Time format [10:00 AM])
  • Progress Bar with percentage
  • Status Text (Done, Cancelled, Processing)
  • Button (returns batch.id for now) [To be updated]

[UPDATED] Refresh Button [Fetch batches]
[NEW] Batch Stage Multiselect Filter Bar

[REMOVED] Bird Eye View on DataImport.py

  • Fetched both processed and unprocessed batches
  • Progress Circle with percentage % shown
  • Count for total of MM & HF

New Batch & Upload Button [To be fixed]

  • Working upload & confirm API

[To be Fixed]: UI Layout

@ZyuT0h ZyuT0h requested a review from Prakhar896 August 1, 2025 20:32
@ZyuT0h ZyuT0h self-assigned this Aug 1, 2025
@ZyuT0h ZyuT0h added the enhancement New feature or request label Aug 1, 2025
Prakhar896
Prakhar896 requested changes Aug 4, 2025
View reviewed changes
Copy link
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

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

Overall, really well done! Just some small fixes to make.

@ZyuT0h ZyuT0h requested a review from Prakhar896 August 9, 2025 17:12
Prakhar896
Prakhar896 requested changes Aug 10, 2025
View reviewed changes
Copy link
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

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

Great work implementing the multi-select filter. Some small issues to fix.

@ZyuT0h ZyuT0h requested a review from Prakhar896 August 11, 2025 06:11
Prakhar896
Prakhar896 approved these changes Aug 11, 2025
View reviewed changes
Copy link
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

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

LGTM

@Prakhar896 Prakhar896 merged commit 7fea7f4 into main Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Prakhar896 Prakhar896 Prakhar896 approved these changes

Assignees

@ZyuT0h ZyuT0h

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ZyuT0h @Prakhar896