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

Introduce the index selector (non-negative) into the recursive engine #62

Closed
V0ldek opened this issue Dec 3, 2022 · 2 comments
Closed

Introduce the index selector (non-negative) into the recursive engine #62

V0ldek opened this issue Dec 3, 2022 · 2 comments
Assignees
@zwerdlds
Labels
area: selector Support for a JSONPath selector contribute: simd Requires SIMD knowledge type: feature New feature or request
Milestone
v1.0.0

Comments

@V0ldek
Copy link
Member

V0ldek commented Dec 3, 2022

Is your feature request related to a problem? Please describe.
After compiling in #61, the execution engines need to handle the new automaton transitions. Start with the recursive engine.

Describe the solution you'd like
When the selector [n] is encountered, the recursive engine should count down elements and mark the n-th as the match.
@V0ldek V0ldek added the type: feature New feature or request label Dec 3, 2022
@github-actions github-actions bot added the acceptance: triage Waiting for owner's input label Dec 3, 2022
@github-actions
Copy link

github-actions bot commented Dec 3, 2022

Tagging @V0ldek for notifications

@V0ldek V0ldek added this to the v1.0.0 milestone Dec 3, 2022
@github-actions github-actions bot added acceptance: go ahead Reviewed, implementation can start and removed acceptance: triage Waiting for owner's input labels Dec 3, 2022
@V0ldek V0ldek mentioned this issue Dec 3, 2022
Closed
@V0ldek V0ldek added mod: engine contribute: simd Requires SIMD knowledge area: selector Support for a JSONPath selector labels Dec 3, 2022
@V0ldek V0ldek mentioned this issue Dec 3, 2022
Closed
4 tasks
@zwerdlds zwerdlds self-assigned this Apr 20, 2023
@V0ldek V0ldek moved this from Todo to Committed in Active rsonpath development Apr 20, 2023
@zwerdlds zwerdlds moved this from Committed to In Progress in Active rsonpath development Apr 20, 2023
@V0ldek
Copy link
Member Author

V0ldek commented May 22, 2023

Completed in #132

@V0ldek V0ldek closed this as completed May 22, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Merged in Active rsonpath development May 22, 2023
@github-actions github-actions bot removed the acceptance: go ahead Reviewed, implementation can start label May 22, 2023
@V0ldek V0ldek moved this from Merged to Released in Active rsonpath development Aug 2, 2023
@V0ldek V0ldek moved this from Todo to Released in Active rq development Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zwerdlds zwerdlds

Labels
area: selector Support for a JSONPath selector contribute: simd Requires SIMD knowledge type: feature New feature or request
Projects
Active rq development
Status: Released
Milestone
v1.0.0
Development

When branches are created from issues, their pull requests are automatically linked.

62-introduce-the-index-selector-non-negative-into-the-recursive-engine zwerdlds/rsonpath
2 participants
@zwerdlds @V0ldek