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

Generalize Containers.h #862

Merged
merged 2 commits into from
Jul 18, 2023
Merged

Generalize Containers.h #862

merged 2 commits into from
Jul 18, 2023

Conversation

KateUnger
Copy link
Collaborator

Description of changes:

Generalize Containers.h to use iterator types

Related Issues:

Linked Issues:

  • Issue #

Issues closed by this PR:

  • Closes #

Before merging:

  • Did you update the flexflow-third-party repo, if modifying any of the Cmake files, the build configs, or the submodules?

@goliaro goliaro added the repo-refactor Topics related to the repo and search refactors label Jul 18, 2023
@lockshaw lockshaw self-requested a review July 18, 2023 21:10
@lockshaw lockshaw merged commit fc8d32a into flexflow:repo-refactor Jul 18, 2023
14 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lockshaw lockshaw lockshaw approved these changes

Assignees
No one assigned
Labels
repo-refactor Topics related to the repo and search refactors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@KateUnger @lockshaw @goliaro