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

Make force-distance segments integer-valued (unit8 or less) #15

Closed
paulmueller opened this issue Jul 7, 2021 · 1 comment
Closed

Make force-distance segments integer-valued (unit8 or less) #15

paulmueller opened this issue Jul 7, 2021 · 1 comment

Comments

@paulmueller
Copy link
Member

paulmueller commented Jul 7, 2021

Because this would make things more consistent with other imaging modalities.

  • Does that break the .tab file format loader? (Yes, needed to adjust parser)
  • Does that break the .hdf5 file format loader? (No, data were stored as integer anyway)
  • Does it break nanite? (Yes, in several places where segment was used for indexing)
  • Does it break nanite rating? (Yes, kind of, workaround was to map "approach" and "retract" to integers to be backwards-compatible)
  • Does it break PyJibe? (Not really)
@paulmueller
Copy link
Member Author

This leads to a breaking change, since nanite uses the "semgent" column for array indexing.

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

1 participant