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

BigTableData Design. #11754

Closed
OwenArnold opened this issue Jan 16, 2015 · 1 comment
Closed

BigTableData Design. #11754

OwenArnold opened this issue Jan 16, 2015 · 1 comment
Assignees
@OwenArnold
Labels
High Priority An issue or pull request that if not addressed is severe enough to postponse a release. Reflectometry Issues and pull requests related to reflectometry

Comments

@OwenArnold
Copy link
Contributor

This issue was originally TRAC 10915

BigTableData is a runtime accident waiting to happen. We think it would be better to have a formal declared type for this complex and critical data structure.

Create a new design document (markdown) and put it in the existing Documents/Design/quick_nxs directory of the documents repository.

  • Outline why this work is important
  • Outline what the existing BigTableData does and stores.
  • Detail what the new type would be called, what it's structure is, and what module it would live in
  • Think about testability of the new structure
  • Think about how the BigTableData replacement would be eased in to the codebase
  • Share the design with Owen
  • Share the design with Peter and Jean
@OwenArnold OwenArnold added High Priority An issue or pull request that if not addressed is severe enough to postponse a release. Reflectometry Issues and pull requests related to reflectometry labels Jun 3, 2015
@OwenArnold OwenArnold added this to the Release 3.5 milestone Jun 3, 2015
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Update class source according to clang format
c0f2d99
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Return string + clang format
bc0fe7c
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Header clang format
6a7064a
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Clang format to unit test
e68a651
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Remove test classes from make file
876349e
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #11754 Add reminders
50ff5e8
@eXeC64 eXeC64 assigned OwenArnold and unassigned eXeC64 Jun 24, 2015
@NickDraper NickDraper modified the milestones: Release 3.5, Release 3.6 Sep 14, 2015
@NickDraper NickDraper modified the milestone: Release 3.6 Jan 22, 2016
@raquelalvarezbanos
Copy link
Contributor

Archived for the same reason as #10847

@raquelalvarezbanos raquelalvarezbanos mentioned this issue Feb 3, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OwenArnold OwenArnold

Labels
High Priority An issue or pull request that if not addressed is severe enough to postponse a release. Reflectometry Issues and pull requests related to reflectometry
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@eXeC64 @NickDraper @OwenArnold @raquelalvarezbanos