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

Semi-deterministic plugin IDs #260

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Semi-deterministic plugin IDs #260

wants to merge 1 commit into from

Conversation

anders-kiaer
Copy link
Collaborator

@anders-kiaer anders-kiaer commented Aug 5, 2020 •
edited
Loading

Closes #223.

  • Implement semi-deterministic plugin IDs.
  • Update contribution docs.

@anders-kiaer anders-kiaer force-pushed the semideterministic_id branch 2 times, most recently from 6592b4c to 66e8a6c Compare August 5, 2020 12:11
@anders-kiaer anders-kiaer mentioned this pull request Aug 5, 2020
Closed
@anders-kiaer anders-kiaer self-assigned this Aug 5, 2020
@anders-kiaer anders-kiaer marked this pull request as ready for review August 5, 2020 12:13
@anders-kiaer
Copy link
Collaborator Author

As of now, Dash components (including dash-core-components) do not check that a persisted/stored value is among updated valid choices. Until this issue is resolved upstream, we should probably be conservative and do what we currently do (change plugin/component IDs on each app restart).

@anders-kiaer anders-kiaer added the blocked ⏳ Blocked by some other software label Aug 22, 2020
@anders-kiaer anders-kiaer marked this pull request as draft September 9, 2020 20:09
@anders-kiaer anders-kiaer mentioned this pull request Jan 30, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asnyv asnyv Awaiting requested review from asnyv

@HansKallekleiv HansKallekleiv Awaiting requested review from HansKallekleiv

At least 1 approving review is required to merge this pull request.

Assignees

@anders-kiaer anders-kiaer

Labels
blocked ⏳ Blocked by some other software
Projects
Webviz
Status: On Hold ✋
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Persisting user preferences & control values
1 participant
@anders-kiaer