Skip to content

Store the history without the conflicting overrides#449

Merged
ps2 merged 1 commit into
LoopKit:devfrom
bjorkert:dev
Feb 16, 2023
Merged

Store the history without the conflicting overrides#449
ps2 merged 1 commit into
LoopKit:devfrom
bjorkert:dev

Conversation

@bjorkert

@bjorkert bjorkert commented Feb 3, 2023

Copy link
Copy Markdown
Contributor

Overlapping overrides was filtered out, but the cleaned list was never stored/persisted before exiting.

Next time loop starts, it will crash with the same error.

@bjorkert

bjorkert commented Feb 3, 2023

Copy link
Copy Markdown
Contributor Author

Relates to LoopKit/Loop#1763

@ps2

ps2 commented Feb 16, 2023

Copy link
Copy Markdown
Collaborator

I'm a little torn between leaving this out so we have a better chance of replicating it, and fixing the underlying bug that causes it. But I'm going to merge it, as it seems rare, and could help prevent someone from getting in a bad situation if they do encounter it.

@ksilive

ksilive commented Mar 14, 2023

Copy link
Copy Markdown

thanks, solved my loop crashing indefinitely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants