utils.box orientation not used in code

Bug #1111514 reported by Alexander Eulitz [Eugen]
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Yade
Fix Released
Wishlist
Bruno Chareyre

Bug Description

Working on Christian Jakobs performance script published int he wiki[1] Christian and I recognized that the orientation quaternion used as third parameter of utils.box[2] is not used at all in the code.
According to Christian this must be due to a change in code that happened in the last months, because his script worked will before.
How can you know include a box that is rotated by a given angle around an axis?

[1] https://yade-dem.org/wiki/Comparisons_with_PFC3D
[2] https://yade-dem.org/doc/_modules/yade/utils.html#box

Revision history for this message
Christian Jakob (jakob-ifgt) wrote :

> According to Christian this must be due to a change in code that happened in the last months, because his script worked will before.

I am not sure about this, maybe I used a facet for the screenshot at that time ...

Nevertheless I have to update the wiki page.

> How can you know include a box that is rotated by a given angle around an axis?

As far as I know it is not possible to rotate boxes.

Revision history for this message
Anton Gladky (gladky-anton) wrote :

Although "orientation" parameter is in utils.box, it is really not used there.
You can use geom.facetBox, which respects "orientation"-parameter.

Anton

Changed in yade:
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Alexander Eulitz [Eugen] (kubeu) wrote :

ok, that will do.

Revision history for this message
Bruno Chareyre (bruno-chareyre) wrote :

>How can you know include a box that is rotated by a given angle around an axis?

q=Quaternion()
q.setFromTwoVectors((1,1,0),(1,0,0))
b.state.ori=q

>the orientation quaternion used as third parameter of utils.box[2] is not used at all in the code

Thank you for reporting this bug. I fixed it in 6279eb1.
So this will do exactly the same as above:

box((0,0,0),(1,0.1,0.1),orientation=q))

Changed in yade:
assignee: nobody → Bruno Chareyre (bruno-chareyre)
status: Confirmed → Fix Released
Revision history for this message
Alexander Eulitz [Eugen] (kubeu) wrote :

thats even better! Thanks a lot

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.