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

Array slice selector (forward, positive bounds) [<start>:<end>:<step>] #152

Closed
V0ldek opened this issue Jun 14, 2023 · 1 comment · Fixed by #495
Closed

Array slice selector (forward, positive bounds) [<start>:<end>:<step>] #152

V0ldek opened this issue Jun 14, 2023 · 1 comment · Fixed by #495
Assignees
Labels
area: selector Support for a JSONPath selector type: feature New feature or request
Milestone

Comments

@V0ldek
Copy link
Member

V0ldek commented Jun 14, 2023

Is your feature request related to a problem? Please describe.
Implementation of the slice selector [<start>:<end>:<step>] for positive steps and positive bounds.

Describe the solution you'd like
Slice for positive steps should be similar to the regular index selector. There are no extraordinary challenges, storing the current index as part of the state should suffice (we already do this for index selectors), and the output is in regular order.

Additional context
RFC draft

@V0ldek V0ldek added the type: feature New feature or request label Jun 14, 2023
@github-actions github-actions bot added the acceptance: triage Waiting for owner's input label Jun 14, 2023
@github-actions
Copy link

Tagging @V0ldek for notifications

@V0ldek V0ldek changed the title Array slice selector Array slice selector (forward, positive bounds) [<start>:<end>:<step>] Jun 14, 2023
@V0ldek V0ldek added this to the v1.0.0 milestone Jun 15, 2023
@github-actions github-actions bot added acceptance: go ahead Reviewed, implementation can start and removed acceptance: triage Waiting for owner's input labels Jun 15, 2023
@V0ldek V0ldek added mod: engine area: selector Support for a JSONPath selector labels Jun 15, 2023
@V0ldek V0ldek mentioned this issue Jan 15, 2024
3 tasks
@V0ldek V0ldek moved this from Todo to In Progress in Active rsonpath development Feb 5, 2024
@V0ldek V0ldek self-assigned this Mar 17, 2024
V0ldek added a commit that referenced this issue Mar 27, 2024
- Simple slicing: forward step and positive bounds.
Includes an overhaul to how array transitions are compiled.

Ref: #152
V0ldek added a commit that referenced this issue Mar 27, 2024
- Simple slicing: forward step and positive bounds.
Includes an overhaul to how array transitions are compiled.

Ref: #152
@V0ldek V0ldek mentioned this issue Mar 27, 2024
3 tasks
V0ldek added a commit that referenced this issue Mar 27, 2024
- Simple slicing: forward step and positive bounds.
Includes an overhaul to how array transitions are compiled.

Ref: #152
V0ldek added a commit that referenced this issue Mar 27, 2024
- Simple slicing: forward step and positive bounds.
Includes an overhaul to how array transitions are compiled.

Ref: #152
@github-project-automation github-project-automation bot moved this from In Progress to Merged in Active rsonpath development Mar 27, 2024
@github-actions github-actions bot removed the acceptance: go ahead Reviewed, implementation can start label Mar 27, 2024
@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
Labels
area: selector Support for a JSONPath selector type: feature New feature or request
Projects
Status: Released
Development

Successfully merging a pull request may close this issue.

1 participant