Skip to content

T4443 followup Restrict Dependency to only include keys in the downward dependency closure #4609

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

soulomoon
Copy link
Collaborator

@soulomoon soulomoon commented Jun 1, 2025

WIP/ Not ready for review
Restrict Dependency to only include keys in the downward dependency closure. As in #4594 (comment)

@soulomoon soulomoon changed the title T4443 followup T4443 followup Restrict Dependency to only include keys in the downward dependency closure. Jun 1, 2025
@soulomoon soulomoon added the performance Issues about memory consumption, responsiveness, etc. label Jun 1, 2025
…aph and filterDependencyInformationReachable functions
@soulomoon soulomoon changed the title T4443 followup Restrict Dependency to only include keys in the downward dependency closure. T4443 followup Restrict Dependency to only include keys in the downward dependency closure. (WIP/ Not ready for review ) Jun 3, 2025
@soulomoon soulomoon marked this pull request as ready for review June 3, 2025 10:47
@soulomoon soulomoon requested a review from wz1000 as a code owner June 3, 2025 10:47
@fendor fendor marked this pull request as draft June 3, 2025 11:54
@fendor fendor changed the title T4443 followup Restrict Dependency to only include keys in the downward dependency closure. (WIP/ Not ready for review ) T4443 followup Restrict Dependency to only include keys in the downward dependency closure Jun 3, 2025
@fendor
Copy link
Collaborator

fendor commented Jun 3, 2025

@soulomoon I turned the PR into a draft, could you mark it ready for review once you are satisfied?

@soulomoon
Copy link
Collaborator Author

@soulomoon I turned the PR into a draft, could you mark it ready for review once you are satisfied?

Yes, sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Issues about memory consumption, responsiveness, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants