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

[performance][wip] Add smart propagation prevention #98

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[performance][wip] Add smart propagation prevention #98

wants to merge 4 commits into from

Conversation

JanKoelzer
Copy link
Contributor

This fixes #82: Do not inform dependent cells about updates, if another update is in sight.

phaller
phaller reviewed Feb 26, 2018
View reviewed changes
Copy link
Owner

@phaller phaller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to see an example (or a class of examples) where this optimization would help.

@JanKoelzer JanKoelzer changed the title Add smart propagation prevention [performance] Add smart propagation prevention Mar 21, 2018
@JanKoelzer JanKoelzer mentioned this pull request Mar 22, 2018
Open
@JanKoelzer
(wip) Add smart propagation prevention – work on errors
ee078d6 
(cherry picked from commit d9c0aab)
@JanKoelzer
(wip) Add smart propagation prevention – work on errors. Maybe this i…
644e370 
…s a fix, but Cell#775 is suspicious.

(cherry picked from commit 584de07)
@JanKoelzer
Fixed the suspicious code.
5dbbd41 
(cherry picked from commit ff4e0a1)
@JanKoelzer
Fix rebase issues.
93a829f
@JanKoelzer JanKoelzer changed the title [performance] Add smart propagation prevention [performance][wip] Add smart propagation prevention Mar 31, 2018
@JanKoelzer JanKoelzer mentioned this pull request Mar 31, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phaller phaller phaller left review comments

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

Successfully merging this pull request may close these issues.

Add "Smart Progagation Prevention"
2 participants
@JanKoelzer @phaller