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

Move SegArray to Server #1570

Closed
2 of 4 tasks
Ethan-DeBandi99 opened this issue Jul 11, 2022 · 2 comments
Closed
2 of 4 tasks

Move SegArray to Server #1570

Ethan-DeBandi99 opened this issue Jul 11, 2022 · 2 comments
Assignees

Comments

@Ethan-DeBandi99
Copy link
Contributor

Ethan-DeBandi99 commented Jul 11, 2022

SegArray is currently a client side construct that keeps track of several pdarray objects to manage its elements. This is very similar to ak.String which is currently referred to as SegString/SegArray on the server. This will eventually be transitioned to SegString on the server entirely as it will most likely inherit functionality from the SegArray module.

This issue is to serve as an overall tracking issue for the transition.

@stress-tess
Copy link
Member

I wanted to link this PR where we first added SegString as a standalone object in chapel (vs two pdarrays that we logically treated as related)

I'm not sure how useful it will be but the PR description should give a good reference for our Hierarchy (CompositeSymEntry is where this would go i'm guessing). And maybe src/MultiTypeSymEntry.chpl will provide an example of sorts?

@Ethan-DeBandi99
Copy link
Contributor Author

The SegArray server side implementation has been reverted to client side only due to functionality limitations due to the server side implementation. Closing issue.

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

No branches or pull requests

2 participants