Skip to content

feat: replace numeric table assignment with dynamic two-floor lettere…#426

Merged
michelleyeoh merged 18 commits intomainfrom
381-update-table-number-to-letter-number
Mar 27, 2026
Merged

feat: replace numeric table assignment with dynamic two-floor lettere…#426
michelleyeoh merged 18 commits intomainfrom
381-update-table-number-to-letter-number

Conversation

@jackzheng-cs
Copy link
Copy Markdown
Contributor

Summary

@jackzheng-cs jackzheng-cs linked an issue Mar 5, 2026 that may be closed by this pull request
@michelleyeoh michelleyeoh requested a review from Copilot March 24, 2026 04:25
Copy link
Copy Markdown

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

Updates CSV ingestion table assignment to generate two-floor letter-number table labels (e.g., A3) and prioritize “Best Hardware Hack” teams on floor 1, aligning with Issue #381’s new table labeling scheme.

Changes:

  • Added a two-floor table assignment algorithm that distributes teams across lettered rows.
  • Prioritized hardware teams on floor 1 and filled remaining floor 1 capacity with non-hardware teams.
  • Replaced the previous sequential numeric table numbering assignment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 13 out of 13 changed files in this pull request and generated 10 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@michelleyeoh michelleyeoh merged commit 6937f65 into main Mar 27, 2026
2 checks passed
@michelleyeoh michelleyeoh deleted the 381-update-table-number-to-letter-number branch March 27, 2026 06:05
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.

Update table number to letter-number

3 participants