Skip to content

Conversation

@angelawu236
Copy link
Contributor

@angelawu236 angelawu236 commented Jan 16, 2026

Summary

Integrate TritonGPT https://github.com/UCSD/campus-mobile/tree/cma-tgpt with existing code from experimental branch.

Changelog

[General] [Add] - Copy and added all changed files from the cma-tgpt branch by @NeuraByte-UCSD-ITS
NOTE: In this current code, tgpt does not function unless auth token is set.

Test Plan

@github-actions
Copy link

Code Style Violations Found

  • Very Long If Statement Conditions: 11 violations

Total Violations: 11
Total Categories Failed: 1/6

Here are the formatting rules to follow:

  • Static Constants have to use UPPER_SNAKE_CASE (e.g., MAX_RETRY_COUNT)
  • Classes have to use UpperCamelCase (e.g., UserProfile)
  • Variables/Functions have to use lowerCamelCase (e.g., userName)
  • Files/Directories have to use snake_case (e.g., user_profile.dart)
  • Imports have to use full package paths (e.g., package:app_name/path/file.dart)
  • If Conditions have to split long/complex conditions into variables for readability

Example of a long if condition violation:

if (userDataProvider.isLoggedIn &&
    (userDataProvider.userProfileModel.classifications?.staff ?? false)) {
  // action
}
// GOOD: split condition into variables for readability
var isLoggedIn = userDataProvider.isLoggedIn;
var isStaff = userDataProvider.userProfileModel.classifications?.staff ?? false;
if (isLoggedIn && isStaff) {
  // action
}

To see more details and fix these violations:

  1. Run ./scripts/auto_check_all.sh locally.
  2. Apply the suggested corrections.
  3. Re-run the script to verify fixes.
  4. Commit your changes.

This comment will continue to appear until all violations are resolved.

@NeuraByte-UCSD-ITS NeuraByte-UCSD-ITS self-assigned this Jan 16, 2026
@NeuraByte-UCSD-ITS
Copy link
Contributor

Hold merge onto experimental branch, this will cause merge issues with the latest cma-tgpt code

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