Skip to content

Conversation

@AmberLJC
Copy link
Owner

@AmberLJC AmberLJC commented Dec 13, 2025

Changes

  • Add DS SERVE paper to Agent Systems section: A Framework for Efficient and Scalable Neural Retrieval from UCB
  • Add prominent NeurIPS 2025 ML Systems Papers section to main README with links to all categorized collections
  • Add NeurIPS 2025 reference to Table of Contents

Closes #42


Generated with Claude Code


Note

Adds a NeurIPS 2025 ML Systems Papers section with categorized links and includes the DS SERVE paper under Agent Systems.

  • Docs (README.md):
    • NeurIPS 2025 ML Systems Papers: Add a new section with links to categorized collections (neurips25-mlsys/*) and update the Table of Contents.
    • Agent Systems: Add DS SERVE entry (efficient, scalable neural retrieval | UCB).

Written by Cursor Bugbot for commit 36bb21c. This will update automatically on new commits. Configure here.

- Add DS SERVE paper to Agent Systems section: A Framework for Efficient and Scalable Neural Retrieval from UCB
- Add prominent NeurIPS 2025 ML Systems Papers section to main README with links to all categorized collections
- Add NeurIPS 2025 reference to Table of Contents

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Jiachen LIU <AmberLJC@users.noreply.github.com>
@AmberLJC
Copy link
Owner Author

@claude move NeurIPS 2025 ML Systems Papers after Industrial LLM Technical Report section. like under ML conferences - NeurIPS 2025

@claude
Copy link

claude bot commented Dec 13, 2025

Claude finished @AmberLJC's task in 1m 34s —— View job


Moving NeurIPS 2025 Section ✓

Tasks:

  • Read README.md to understand current structure
  • Move NeurIPS 2025 ML Systems Papers section after Industrial LLM Technical Report
  • Format as "ML conferences - NeurIPS 2025" subsection
  • Update Table of Contents if needed
  • Commit and push changes

Changes completed:

  • Removed NeurIPS 2025 section from the top of README
  • Created new "ML Conferences" section after Industrial LLM Technical Report
  • Added NeurIPS 2025 as a subsection under ML Conferences
  • Updated Table of Contents with proper hierarchy

The NeurIPS 2025 ML Systems Papers section has been successfully reorganized as requested!

--- |

@claude
Copy link

claude bot commented Dec 13, 2025

Code Review Summary

✅ Overall Assessment

This PR successfully adds the DS SERVE paper and creates a prominent reference section for NeurIPS 2025 ML Systems papers. The changes are well-structured and follow the repository conventions outlined in CLAUDE.md.


📋 Detailed Review

Formatting & Style

  • NeurIPS 2025 Section: Well-formatted with clear organization and proper markdown structure
  • DS SERVE Entry: Follows the correct format: [Title](URL): Description | Organization
  • Table of Contents: Properly updated to include the new section
  • Consistency: Maintains consistency with existing entries in the repository

Content Quality

  • Placement: DS SERVE is correctly placed in the Agent Systems subsection (README.md:262)
  • Description: Clear and accurate description matching the paper's contribution
  • Links: All internal links to neurips25-mlsys/*.md files are valid and accessible
  • Organization: NeurIPS 2025 section is appropriately positioned at the top for high visibility

Potential Issues ⚠️

  1. Non-arXiv Link for DS SERVE

    • The DS SERVE entry uses a project website (https://berkeley-large-rag.github.io/RAG-DS-Serve/) instead of an arXiv link
    • Most other entries in the repository use arXiv links when available
    • Recommendation: If an arXiv paper exists for DS SERVE, consider using that link instead for consistency
    • If no arXiv paper is available, the project website is acceptable (similar to SOSP entries using conference links)
  2. Section Positioning

    • The NeurIPS 2025 section is placed before "LLM Systems" which breaks the traditional flow
    • This is a design choice rather than an error - it gives high visibility to recent conference papers
    • Consider: Whether this should be a permanent top-level section or eventually integrated into the main taxonomy

🔍 Code Quality Checklist

  • ✅ No syntax errors or markdown formatting issues
  • ✅ All file references are valid
  • ✅ Table of Contents correctly updated
  • ✅ No duplicate entries
  • ✅ Follows repository naming conventions
  • ✅ Proper use of headers (##, ###, ####)
  • ✅ Consistent spacing and formatting

🔐 Security & Performance

  • ✅ No security concerns (documentation only)
  • ✅ No performance impact (static markdown files)
  • ✅ External links point to legitimate academic/institutional domains

📝 Suggestions for Improvement

  1. Consider adding arXiv link for DS SERVE (if available):

    - [DS SERVE](https://arxiv.org/abs/XXXX.XXXXX): A Framework for Efficient and Scalable Neural Retrieval | UCB
  2. Optional: Add a brief note in the NeurIPS section explaining its prominent placement:

    ## NeurIPS 2025 ML Systems Papers
    
    **Featured collection of recent systems research from NeurIPS 2025.**

✅ Approval Recommendation

APPROVED - This PR is well-executed and ready to merge. The only minor suggestion is to verify if an arXiv link exists for DS SERVE for better consistency with other entries.

Great work on organizing the NeurIPS 2025 papers into a comprehensive, well-categorized collection! The addition enhances the repository's value as a living resource for ML systems research.


Review generated with attention to repository guidelines in CLAUDE.md

…nical Report

- Created new "ML Conferences" section after Industrial LLM Technical Report
- Moved NeurIPS 2025 ML Systems Papers as subsection under ML Conferences
- Updated Table of Contents with proper hierarchy

Co-authored-by: Jiachen LIU <AmberLJC@users.noreply.github.com>
@claude
Copy link

claude bot commented Dec 13, 2025

Pull Request Review

Thank you for contributing to the LLMSys-PaperList repository! I've reviewed PR #43 and have the following feedback:

✅ Positive Observations

  1. Proper Table of Contents Update: The ToC was correctly updated to include the new ML Conferences section with the NeurIPS 2025 subsection.

  2. Good Section Placement: The new ML Conferences section is logically placed after Industrial LLM Technical Report and before LLM Frameworks, which makes sense organizationally.

  3. Comprehensive NeurIPS 2025 Content: The neurips25-mlsys directory contains well-organized categorized papers with proper documentation.

  4. Clear Descriptions: The NeurIPS 2025 section provides clear descriptions of what each category contains.


⚠️ Issues and Recommendations

1. Inconsistent Formatting for DS SERVE Entry

Issue: Line 251 uses a project website URL instead of an arXiv/paper URL, which is inconsistent with the repository's formatting guidelines.

- [DS SERVE](https://berkeley-large-rag.github.io/RAG-DS-Serve/): A Framework for Efficient and Scalable Neural Retrieval | UCB

Problem: According to CLAUDE.md guidelines, papers should use arXiv links (format: https://arxiv.org/abs/XXXX.XXXXX) or official conference/journal URLs. Project websites are not the standard format for paper entries.

Recommendation:

  • If DS SERVE has an arXiv link or published paper URL, use that instead
  • If it's only a project website with no paper, consider adding it to a different section or clarifying its status
  • Check if there's a technical report or publication associated with this work

Suggested fix (if arXiv paper exists):

- [DS SERVE](https://arxiv.org/abs/XXXX.XXXXX): A Framework for Efficient and Scalable Neural Retrieval | UCB

2. Section Title Concerns

Issue: The new section is titled ## ML Conferences but currently only contains NeurIPS 2025.

Potential Issues:

  • This creates an expectation that other conferences will be added (e.g., ICML, ICLR, SOSP, etc.)
  • If the plan is only for NeurIPS, a more specific title might be better
  • May require maintenance as more conferences are added

Recommendations (choose one approach):

  • Option A: Rename to ## Conference Collections or ## Curated Conference Papers to be more specific
  • Option B: Keep the current name but add a comment about the plan for other conferences
  • Option C: Consider if this should be integrated differently (e.g., under a "Special Collections" section)

3. Chronological Ordering

Note: The DS SERVE entry appears to be added at line 251, which is chronologically ordered. Verify this is the intended position based on publication date.


📝 Code Quality Assessment

  • Syntax: All markdown syntax is correct
  • Links: Internal links to neurips25-mlsys/* files are properly formatted
  • Consistency: Most formatting follows the repository style guide
  • ⚠️ URL Format: DS SERVE uses non-standard URL format (see issue feature request: Add conference name #1)
  • Organization: Logical section placement and structure

🔒 Security & Performance

  • ✅ No security concerns (documentation only)
  • ✅ No performance implications (static content)

🧪 Testing Considerations

Manual verification needed:

  1. Verify all internal links work: neurips25-mlsys/architecture.md, etc.
  2. Verify the DS SERVE URL is accessible and correct
  3. Check if DS SERVE has a proper paper publication that should be linked instead

📊 Summary

Must Fix:

  1. DS SERVE URL format - needs to use arXiv or official publication link per guidelines

Should Consider:
2. Section naming - is ML Conferences the right name for just NeurIPS?

Overall Assessment: Good contribution that adds valuable content, but needs one formatting fix to align with repository guidelines. The NeurIPS 2025 collection is comprehensive and well-organized.


Action Items:

  1. Find and use the proper paper URL for DS SERVE (arXiv or conference link)
  2. Consider renaming the ML Conferences section if it will remain NeurIPS-only
  3. After addressing issue feature request: Add conference name #1, this PR will be ready to merge ✅

@AmberLJC AmberLJC merged commit be32727 into main Dec 13, 2025
1 check passed
@AmberLJC AmberLJC deleted the claude/issue-42-20251212-2352 branch December 13, 2025 01: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.

DS SERVE + neurips25-mlsys

2 participants