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

Should the barcode be included or not? #3

Open
eriksjolund opened this issue Jun 15, 2016 · 0 comments
Open

Should the barcode be included or not? #3

eriksjolund opened this issue Jun 15, 2016 · 0 comments
Labels

Comments

@eriksjolund
Copy link
Owner

Right now the barcode is included in the st_exp_protobuf format.
Maybe we should leave it out of the format?

To remove it, the file st_exp.proto would have to be modified here:

message Spot {
  // We limit the total number of spots to (2^32-1) as the spot id is uint32_t.
  // Therefore we limit the grid coordinates to be max (2^16-1).
  required uint32 xCoordGrid = 1;  // integer in range: 0 ... (2^16-1) 
  required uint32 yCoordGrid = 2;  // integer in range: 0 ... (2^16-1)
  required float xCoordPhyscial = 3;  // millimeter
  required float yCoordPhyscial = 4;  // millimeter
  required string barcode = 5;
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant