Skip to content
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

Update default FPGA part number from KU115 to VU13P #924

Merged
merged 7 commits into from Nov 15, 2023
Merged

Update default FPGA part number from KU115 to VU13P #924

merged 7 commits into from Nov 15, 2023

Conversation

jmduarte
Copy link
Member

Description

  • Update default FPGA part number from KU115 to VU13P, as more commonly used by hls4ml users (e.g., for CMS Phase-2 Level-1 Trigger)

Type of change

  • Documentation update
  • Other: Default configuration change

Tests

  • Checking in CI

Checklist

  • I have read the guidelines for contributing.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have installed and run pre-commit on the files I edited or added.
  • I have added tests that prove my fix is effective or that my feature works.

@jmitrevs jmitrevs merged commit d9391dd into main Nov 15, 2023
5 checks passed
@jmitrevs jmitrevs deleted the vu13p branch November 15, 2023 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmitrevs jmitrevs jmitrevs approved these changes

@vloncar vloncar Awaiting requested review from vloncar

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jmduarte @jmitrevs @vloncar