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

Support slicing of ACAReviewTable #151

Merged
merged 1 commit into from
Apr 1, 2021
Merged

Support slicing of ACAReviewTable #151

merged 1 commit into from
Apr 1, 2021

Conversation

taldcroft
Copy link
Member

Description

This fixes #150 by allowing creation of an empty ACAReviewTable which can then be constructed by the normal Table slicing machinery. The resultant object is not a full-fledged ACAReviewTable and you cannot run review methods and such on it. I think it would be possible to do this if there were a real driver, but it would require overriding __getitem__ and might get messy.

Testing

  • Passes unit tests on MacOS with new unit test
  • [N/A] Functional testing

Fixes #150

@taldcroft taldcroft added the bug Something isn't working label Mar 13, 2021
@taldcroft taldcroft merged commit c44fa5c into master Apr 1, 2021
This was referenced Jun 30, 2021
@jeanconn jeanconn deleted the support-slicing branch March 2, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't index into a review table
2 participants