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.

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

🔬Research collection conversion performance #486

Open
4 of 6 tasks
CalvinWilkinson opened this issue Jan 6, 2023 · 0 comments
Open
4 of 6 tasks

🔬Research collection conversion performance #486

CalvinWilkinson opened this issue Jan 6, 2023 · 0 comments
Assignees
Labels
🔬research Research collection / investigation

Comments

@CalvinWilkinson
Copy link
Member

CalvinWilkinson commented Jan 6, 2023

Complete The Item Below

  • I have updated the title without removing the 🔬 emoji.

What To Research

Research to see if the performance of List<T>().AsReadOnly() is better than using new ReadOnlyCollection<T>(items).

Very sure that this is the case but the idea is to always measure and prototype the impact before implementing this perf improvement. If the research shows that it is a significant improvement, then create an issue to implement the improvement.

This will for sure reduce allocations. But the entire idea is to research the impact of the change more than anything.

Research Results

No response

Acceptance Criteria

This issue is finished when:

  • Research complete and issues created (if needed).
  • If any issues were created, they have been added to the Related Work section below.

ToDo Items

  • Priority label added to this issue. Refer to the Priority Type Labels section below.
  • Issue linked to the correct project.

Issue Dependencies

No response

Related Work

No response

Additional Information:

Priority Type Labels

Priority Type Label
Low Priority low priority
Medium Priority medium priority
High Priority high priority

Code of Conduct

  • I agree to follow this project's Code of Conduct.
@CalvinWilkinson CalvinWilkinson added high-priority High Priority 🔬research Research collection / investigation labels Jan 6, 2023
@CalvinWilkinson CalvinWilkinson self-assigned this Jan 6, 2023
@github-project-automation github-project-automation bot moved this to ⚪Not Set in KD-Team Jan 6, 2023
@CalvinWilkinson CalvinWilkinson removed the high-priority High Priority label May 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔬research Research collection / investigation
Projects
Status: ⚪Not Set
Development

No branches or pull requests

1 participant