Merge lp:~mflodin/forssim/cylinderfix into lp:forssim
- cylinderfix
- Merge into development
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | not available | ||||
Proposed branch: | lp:~mflodin/forssim/cylinderfix | ||||
Merge into: | lp:forssim | ||||
Diff against target: | None lines | ||||
To merge this branch: | bzr merge lp:~mflodin/forssim/cylinderfix | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonas Forsslund | Approve | ||
Review via email:
|
Commit message
Description of the change
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Martin Flodin (mflodin) wrote : | # |
- 275. By Martin Flodin
-
Finished commenting cylinder.x3d and cleaned out references to ADrillForce and ADrillableNode. Also created a touchable_
cylinder. x3d for use with haptics. - 276. By Martin Flodin
-
Added an example where drilling is enabled via haptics and one via mouse. Also added some pythonfiles for interaction, but there seems to be some problem with drilling.
- 277. By Martin Flodin
-
Drilling works in drillable_
cylinder. x3d. - 278. By Martin Flodin
-
Cleaned up and made sure all examples were consistent. All examples work now, but the mousehaptics is a bit hard to control.
- 279. By Martin Flodin
-
Commented a bit more.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Martin Flodin (mflodin) wrote : | # |
> Fixed one_cylinder so it is able to run, but the code might need more
> comments.
I have commented all the files now.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jonas Forsslund (jonas.forsslund) wrote : | # |
looks good to me /jofo
I like that launchpad shows the diff of the whole branch, including the files added (I assume it does this). That proves the point of using branches instead of only commits, since its hard to review each commit
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jonas Forsslund (jonas.forsslund) wrote : | # |
how does the unmerged revisions work?
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Martin Flodin (mflodin) wrote : | # |
That's another good thing about using branches - the revision history
remains in each branch.
2009/5/14 Jonas Forsslund <email address hidden>:
> how does the unmerged revisions work?
> --
> https:/
> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jonas Forsslund (jonas.forsslund) wrote : | # |
agree. oh, i meant the uncommited revisions, you had some commits
there after the suggested merge right?
will those be ignored or merged in?
Jonas
(sending by replying email!)
2009/5/15 Martin Flodin <email address hidden>:
> That's another good thing about using branches - the revision history
> remains in each branch.
>
> 2009/5/14 Jonas Forsslund <email address hidden>:
>> how does the unmerged revisions work?
>> --
>> https:/
>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>
> --
> https:/
> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Martin Flodin (mflodin) wrote : | # |
Aha. I think they will be merged in. We still have to do the merge
manually (bzr merge lp:~mflodin/forssim/cylinderfix) in the
development directory as far as I understand it. Or can you get
launchpad to do it?
/M
2009/5/15 Jonas Forsslund <email address hidden>:
> agree. oh, i meant the uncommited revisions, you had some commits
> there after the suggested merge right?
> will those be ignored or merged in?
>
> Jonas
> (sending by replying email!)
>
> 2009/5/15 Martin Flodin <email address hidden>:
>> That's another good thing about using branches - the revision history
>> remains in each branch.
>>
>> 2009/5/14 Jonas Forsslund <email address hidden>:
>>> how does the unmerged revisions work?
>>> --
>>> https:/
>>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>>
>> --
>> https:/
>> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>>
> --
> https:/
> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Jonas Forsslund (jonas.forsslund) wrote : | # |
i see
no i did not found it. i guess one does it manually
can you check before next push that your last commit adds to the
"review" diff for
the branch you already have suggested for merge?
or do you need to do a new suggest to merge...?
2009/5/15 Martin Flodin <email address hidden>:
> Aha. I think they will be merged in. We still have to do the merge
> manually (bzr merge lp:~mflodin/forssim/cylinderfix) in the
> development directory as far as I understand it. Or can you get
> launchpad to do it?
>
> /M
>
> 2009/5/15 Jonas Forsslund <email address hidden>:
>> agree. oh, i meant the uncommited revisions, you had some commits
>> there after the suggested merge right?
>> will those be ignored or merged in?
>>
>> Jonas
>> (sending by replying email!)
>>
>> 2009/5/15 Martin Flodin <email address hidden>:
>>> That's another good thing about using branches - the revision history
>>> remains in each branch.
>>>
>>> 2009/5/14 Jonas Forsslund <email address hidden>:
>>>> how does the unmerged revisions work?
>>>> --
>>>> https:/
>>>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>>>
>>> --
>>> https:/
>>> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>>>
>> --
>> https:/
>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>
> --
> https:/
> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Martin Flodin (mflodin) wrote : | # |
Hmm, interesting question. I suppose the review diff at launchpad will
change to reflect the latest revision but the one attached to the mail
naturally won't. But I'll be sure to look into it at the next merge.
2009/5/15 Jonas Forsslund <email address hidden>:
> i see
> no i did not found it. i guess one does it manually
>
> can you check before next push that your last commit adds to the
> "review" diff for
> the branch you already have suggested for merge?
> or do you need to do a new suggest to merge...?
>
> 2009/5/15 Martin Flodin <email address hidden>:
>> Aha. I think they will be merged in. We still have to do the merge
>> manually (bzr merge lp:~mflodin/forssim/cylinderfix) in the
>> development directory as far as I understand it. Or can you get
>> launchpad to do it?
>>
>> /M
>>
>> 2009/5/15 Jonas Forsslund <email address hidden>:
>>> agree. oh, i meant the uncommited revisions, you had some commits
>>> there after the suggested merge right?
>>> will those be ignored or merged in?
>>>
>>> Jonas
>>> (sending by replying email!)
>>>
>>> 2009/5/15 Martin Flodin <email address hidden>:
>>>> That's another good thing about using branches - the revision history
>>>> remains in each branch.
>>>>
>>>> 2009/5/14 Jonas Forsslund <email address hidden>:
>>>>> how does the unmerged revisions work?
>>>>> --
>>>>> https:/
>>>>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>>>>
>>>> --
>>>> https:/
>>>> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>>>>
>>> --
>>> https:/
>>> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>>>
>> --
>> https:/
>> You are reviewing the proposed merge of lp:~mflodin/forssim/cylinderfix into lp:forssim.
>>
> --
> https:/
> You are subscribed to branch lp:~mflodin/forssim/cylinderfix.
>
Preview Diff
1 | === added file 'models/one_cylinder/data/empty.nrrd' |
2 | Binary files models/one_cylinder/data/empty.nrrd 1970-01-01 00:00:00 +0000 and models/one_cylinder/data/empty.nrrd 2009-05-07 15:01:18 +0000 differ |
3 | === added file 'models/one_cylinder/data/partitions.nrrd' |
4 | Binary files models/one_cylinder/data/partitions.nrrd 1970-01-01 00:00:00 +0000 and models/one_cylinder/data/partitions.nrrd 2009-05-07 15:01:18 +0000 differ |
5 | === added file 'models/one_cylinder/data/segmentID.nrrd' |
6 | Binary files models/one_cylinder/data/segmentID.nrrd 1970-01-01 00:00:00 +0000 and models/one_cylinder/data/segmentID.nrrd 2009-05-07 15:01:18 +0000 differ |
7 | === modified file 'models/one_cylinder/x3d/cylinder.x3d' |
8 | --- models/one_cylinder/x3d/cylinder.x3d 2009-03-16 08:07:07 +0000 |
9 | +++ models/one_cylinder/x3d/cylinder.x3d 2009-05-07 15:01:18 +0000 |
10 | @@ -10,30 +10,49 @@ |
11 | <Viewpoint position="0 0 0.25" fieldOfView="0.895040"/> |
12 | <Transform rotation="0.9 0.2 0.3 1.1"> |
13 | |
14 | - <!-- Specifies the material segments we have and their hardness (=1/time needed to carve into material) --> |
15 | + <!-- Specifies the material segments we have and their hardness (=1/time needed to carve into material) |
16 | + The values are automatically set to 0, 1, 2, ... for each segment. The segments are specified in |
17 | + a .nrrd file. Segments are non-overlapping. The number of values in segmentNameField needs to match |
18 | + that of segmentHardnessField. --> |
19 | <MaterialSegmentationModel DEF='SM' |
20 | - segmentIdField = '64 128 192 255' |
21 | - segmentNameField ='"Mandibular Nerve" "Bone" "Tooth bone" "Tooth enamel"' |
22 | - segmentHardnessField = '2.0 0.5 0.3 0.15'> |
23 | + segmentNameField ='"Air" "Mandibular Nerve" "Bone" "Tooth bone" "Tooth enamel"' |
24 | + segmentHardnessField = '0.0 2.0 0.5 0.3 0.15'> |
25 | </MaterialSegmentationModel> |
26 | |
27 | + |
28 | + |
29 | + <!-- PartitionModel enables you to specify different parts of the model with a .nnrd file matching the |
30 | + values in partitionIdField. You can also name your partitions for clarity. Partitions are non-overlapping. |
31 | + Note that each partition can contain any number of segments from MaterialSegmentationModel, and vice versa. |
32 | + The number of values in partitionIDField needs to match the number of values in partitionNameField. --> |
33 | + |
34 | <PartitionModel DEF='PM' |
35 | - partitionIdField = '64 128 192 255' |
36 | - partitionNameField ='"Mandibular Nerve" "Bone" "Tooth bone" "Tooth enamel"'> |
37 | + partitionIdField = '1 2' |
38 | + partitionNameField ='"Top" "Bottom"'> |
39 | </PartitionModel> |
40 | |
41 | + <!-- Can be used to specify parts of the volume that you are not allowed to drill in. |
42 | + Sort of an anti-expert. Needs at least one segment even if it is not used. --> |
43 | + <ForbiddenSegmentationModel DEF='FM' |
44 | + segmentIdField = '1' |
45 | + segmentNameField ='IsNotUsed'> |
46 | + </ForbiddenSegmentationModel> |
47 | + |
48 | + |
49 | |
50 | <LoggingNode DEF='LN'></LoggingNode> |
51 | |
52 | <!-- Our main model, it consists of a visual model + material segment + partition + expert steps--> |
53 | <VolumeModel DEF='VM' |
54 | - materialSegmentationField='../data/cylinder.nrrd' |
55 | - partitionField='../data/cylinder.nrrd' |
56 | - expertDrillingStepsField='../data/cylinder_expert.nrrd'> |
57 | - |
58 | + materialSegmentationField='../data/segmentID.nrrd' |
59 | + partitionField='../data/partitions.nrrd' |
60 | + expertDrillingStepsField='../data/cylinder_expert.nrrd' |
61 | + forbiddenSegmentationField='../data/empty.nrrd' > |
62 | + |
63 | <Image3DTexture containerField="imageField" url="../data/cylinder.nrrd" DEF="visuals" /> |
64 | <MaterialSegmentationModel containerField="materialSegmentationModelField" USE="SM" /> |
65 | <PartitionModel containerField="partitionModelField" USE="PM" /> |
66 | + <ForbiddenSegmentationModel containerField="forbiddenSegmentationModelField" USE="FM" /> |
67 | <LoggingNode containerField="loggingNodeField" USE="LN" /> |
68 | |
69 | </VolumeModel> |
Fixed one_cylinder so it is able to run, but the code might need more comments.