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

feat: bidirectional references #104

Open
wants to merge 190 commits into
base: develop
Choose a base branch
from
Open

Conversation

iammadab
Copy link
Contributor

@iammadab iammadab commented Apr 28, 2022

referenced items should keep track of all elements that reference them
this allows efficient updates to the references when the base item changes (updated or deleted)

When a base item gets deleted all its references should also get deleted.
When a reference gets deleted, the base item should stop pointing to it.

iammadab and others added 30 commits March 10, 2022 07:37
* test range from proof

* add unbounded right

* test for range to proof

* add left unbounded

* add absence tuple checks to test

* test range to inclusive

* test for range after proof

* test for range after proof

* test for range after to inclusive

* test for range full proof

* update verify proof func to account for unbounded ranges

* add verification check for right non inclusive ranges

* refactor bounds to return options

* update verify function to work with range_after_* queries

* fix range_after_proof test

* fix range_after_to_proof test

* fix range_after_to_proof_inclusive test

* fix range_full_proof
@iammadab iammadab marked this pull request as ready for review May 9, 2022 09:39
Base automatically changed from feat/sync-references to develop May 24, 2022 08:30
Base automatically changed from develop to master June 27, 2022 23:36
@QuantumExplorer QuantumExplorer changed the base branch from master to develop July 19, 2022 08:36
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.

2 participants