-
Notifications
You must be signed in to change notification settings - Fork 4
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
Transformation Update #113
Transformation Update #113
Conversation
b542a3a
to
db6d93b
Compare
building now... and will confirm, but a comment on the commit message re: splitting the .tfm: |
Okay, I will update this now to include the {bone}.tfm again |
db6d93b
to
f9fb943
Compare
suggest
|
Should this also apply to the naming of the bone_scale (ie. inv(bone_scale))
Would something like |
d4a3837
to
e89a735
Compare
@@ -757,30 +767,29 @@ def onLoadPV(self): | |||
transformSubDir = self.ui.tfmSubDir.text | |||
|
|||
vols = glob.glob(os.path.join(mainOutputDir, volumeSubDir, "*.tif")) | |||
tfms = glob.glob(os.path.join(mainOutputDir, transformSubDir, "*.tfm")) | |||
tfms_t = glob.glob(os.path.join(mainOutputDir, transformSubDir, "*_t.tfm")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Independently of this pull request, we should consider documentating the different transform files generated by Autoscoper/AutoscoperM.
May be by updating the following section: https://autoscoper.readthedocs.io/en/latest/file-specifications/index.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After addressing few nitpicks and confirming there are no regression, unless @amymmorton has additional comments, this is good for integration 🚀
e89a735
to
fff4cc0
Compare
fff4cc0
to
2a4b20e
Compare
{bone}.tfm
into{bone}_t.tfm
and{bone}_scale.tfm