← Back to team overview

yade-dev team mailing list archive

Re: Block Generation, Contact Detection, Rock Bolt and Rock Lining

 

Hi Boon,
Adding dependencies on third party library really means a dependency, i.e. we have to assume that the said library will be continuously maintained by the third party guys, documented, etc. After a quick look, CLP seems to be a very low-level library, hence weakly sensitive to system upgrades. That is a good point. It is packaged for debian to (you can "sudo apt-get install coinor-clp"). So I don't think it is a big problem to have it; neither a barrier to share. Still a bit more packaging work though. If it is really a problem it could be replaced by Eigen and CGAL but I don't how deeply it would need to change the code (and if that is only possible).

What is the situation now, does it work as expected? Why are you saying it is difficult to share? Currently, is the user supposed to compile his own version of CLP or is it using the debian-packaged version?

Bruno


On 09/04/2016 05:15 PM, Chia Weng Boon wrote:

Hi All,

I have just returned from Eurock 2016, and there is good response on the code.

Thinking back, is it because of the need for an external library CLP that makes sharing it on YADE difficult?

Please let me know how I can make the sharing of the code easier.


Boon

------------------------------------------------------------------------
*From:* Chia Weng Boon on behalf of Chia Weng Boon <chiaweng.boon@xxxxxxxxxxxxxxxx>
*Sent:* Sunday, September 4, 2016 11:15:17 PM
*To:* Jan Stránský; Bruno Chareyre; stefano utili; Yade developers
*Subject:* Re: [Yade-dev] Block Generation, Contact Detection, Rock Bolt and Rock Lining

Hi All,


I have just returned from Eurock 2016, and there is good response on the code.


Thinking back, is it because of the need for an external library CLP that makes sharing it on YADE difficult?


Please let me know how I can make the sharing of the code easier.



Boon



------------------------------------------------------------------------
*From:* Chia Weng Boon <chiaweng@xxxxxxxxx>
*Sent:* Sunday, August 14, 2016 8:09 PM
*To:* Jan Stránský; Bruno Chareyre; stefano utili; chiaweng.boon@xxxxxxxxxxxxxxxx *Subject:* Re: [Yade-dev] Block Generation, Contact Detection, Rock Bolt and Rock Lining
Dear Jan,
I realised that there is going to be a new version of Ubuntu ver 16. I would recommend implementing the shared code onto the current YADE first, before YADE gets updated again. Now what I have shared should compile OK (it was running fine during March 2016). I am just afraid that the code will be incompatible with YADE after it gets updated with a newer version of Ubuntu.

Yours,
Boon

On Sun, Aug 7, 2016 at 11:27 PM, Chia Weng Boon <chiaweng@xxxxxxxxx <mailto:chiaweng@xxxxxxxxx>> wrote:

    Dear Jan,

    Thanks. I have not been in touch with YADE. When I shared the
code, it is already compatible with YADE version in March 2016. If you have any questions please write to
    chiaweng.boon@xxxxxxxxxxxxxxxx <mailto:chiaweng.boon@xxxxxxxxxxxxxxxx>
    Everything should compile fine.

    Let me know if you have any queries. Or is it available in trunk
    already?  I can probably assist with the manual?


    Yours,
    Boon

    On Thu, Mar 31, 2016 at 12:02 AM, Jan Stránský
    <honzik.stransky@xxxxxxxxx <mailto:honzik.stransky@xxxxxxxxx>> wrote:

        Hi Boon,
        thanks for the effort also from me and sorry for late answer.
        What is the situation now? I can help to merge the files not
        to conflict with current branch, but probably not before May..
        Regards
        Jan


        2016-03-17 17:11 GMT+01:00 Chia Weng Boon <chiaweng@xxxxxxxxx
        <mailto:chiaweng@xxxxxxxxx>>:

            Bruno,

            You are absolutely right in all those cases.   ScGeom can
            make use of a boolean for the twist.  Yes, Cundall's paper
            is almost 30 years old.  He may have changed his mind
            too.  I remember the code crashed when I was using the new
            updateProperties.  I am not a good debugger.   And the
            Clump::del should work.

            In fact, there had been some changes in YADE's main
            program that I had to modify my old code, to make it
            work.  I mentioned those few regressions,  so that active
            developers are able to identify them and fix it.  I
            decided that if I were to keep the code and try to update
            it nicely before sharing it out, YADE may get updated
            again, before I can catch up.  I better share it when it
is working. This is my advice for other researchers. Don't keep your code, because you will likely have to keep
            up with the developments in YADE. And you won't have time
            to play/maintain with your own code anyway when you move
            on with life.   Now that I am in construction and not in a
            university, it is very difficult to have the luxury of
            finding time for this hobby.



            Yours,
            Boon

            On Thu, Mar 17, 2016 at 11:01 PM, Bruno Chareyre
            <bruno.chareyre@xxxxxxxxxxxxxxx
            <mailto:bruno.chareyre@xxxxxxxxxxxxxxx>> wrote:

                Hi Boon,
                Thank you for progress report!

                On 16/03/16 16:37, Chia Weng Boon wrote:
                > I have also changed some of the existing files to
                make it compatible with
                > my old files. Please update and modify it
                accordingly if you think there
                > is a better way to do it.
                > ScGeom:  I took away the twist update because for
                polygons this is not used
                > in Cundall's paper.  Please modify accordingly if
                there is some theoretical
                > discovery.
                Does it mean that the twist will be also disabled if
                someone has, let's
                say, a sphere-sphere contact?
                And did you find a reason why the twist should be
                defined differently
                (or even undefined) in the case of polygons?
                In general Yade is not meant to reproduce Cundall's
                equations on every
                aspect.

                > Clump:  The Update Properties looked different from
                the old one, and I had
                > problems with clumps earlier on when updating the
                code.  Not sure if this
                > is the problem, but i used the old one anyway.
                We must make sure that adding some code does not imply
                regressions as
                side effects.
                Do you have any clue why the polyhedra code needs an
                older version of
                updateProperties? What sort of problem do you find?
                Rolling back such a function is most likely breaking,
                or at least
                changing the behavior, of other parts of the code.

                > YadeWrapper: Added deleteClumpMembers function,
                since it was in my old
                > version.
                Is Clump::del not doing the same thing? Do you need
                such a function?

                As far as I understand, you are half-way to a
                compatible code overall.
                It will be really compatible when no changes will be
                required in other
                files.
                If somene else will do the remaining work, it would be
                helpful to him to
                have more detailed explanations of why you had to roll
                back some changes
                (which really means something to fix in your code, not
                something to roll
                back in trunk).

                Cheers

                Bruno



                _______________________________________________
                Mailing list: https://launchpad.net/~yade-dev
                <https://launchpad.net/%7Eyade-dev>
                Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
                <mailto:yade-dev@xxxxxxxxxxxxxxxxxxx>
                Unsubscribe : https://launchpad.net/~yade-dev
                <https://launchpad.net/%7Eyade-dev>
                More help   : https://help.launchpad.net/ListHelp
                <https://help.launchpad.net/ListHelp>



            _______________________________________________
            Mailing list: https://launchpad.net/~yade-dev
            <https://launchpad.net/%7Eyade-dev>
            Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
            <mailto:yade-dev@xxxxxxxxxxxxxxxxxxx>
            Unsubscribe : https://launchpad.net/~yade-dev
            <https://launchpad.net/%7Eyade-dev>
            More help   : https://help.launchpad.net/ListHelp
            <https://help.launchpad.net/ListHelp>





--
_______________
Bruno Chareyre
Associate Professor
ENSE³ - Grenoble INP
Lab. 3SR
BP 53
38041 Grenoble cedex 9
Tél : +33 4 56 52 86 21
Fax : +33 4 76 82 70 43
________________

Email too brief?
Here's why! http://emailcharter.org


Follow ups

References