> (It seems the yade project started using clang-format (or similar
> tool) sometime between e07e530 and 1b4ae97 so `diff` outputs half
> the file: not very helpful… 😆️)

Hi,

reformat with the same .clang-format file your old branch (*). Then
you will be back to meaningful diffs. I used this trick more than
once when fighting old bugs which appeared long before clang-format
and comparing with newer versions: this eliminates all formatting
differences and only meaningful code changes remain.

best regards
Janek

(*) you can use: `scripts/clang-formatter.sh ./` (copied to old
branch, along with .clang-format config file)


Gael Lorieul said:     (by the date of Tue, 23 Feb 2021 20:59:33 +0000)

> Hi 😊️
> 
> It has been a long time…
> 
> I have come accross thread #694548 on the "Yade-users" mailing list (from 
> January 2021), which concerned unphysical collisions between PFacets (the 
> dynamics of the rebound does not correspond to what one's physical/common 
> sense would expect).
> 
> I wanted to share that I encountered a very similar issue but with 
> gridConnection objects (I work with cylinders), and that I found a solution. 
> The latter should be easily extendible both to PFacet/PFacet and 
> PFacet/gridConnection collisions (although I only tested 
> gridConnection/gridConnection collisions). I am not sure how elegant or 
> computationally efficient it is, but it has given me good results on the 
> (very few) simulations I have tested it on.
> 
> I initially wanted to test my solution on full-scale simulations before 
> submiting it to the Yade code base, so as to present a code that is robust 
> and mature. But as it has been almost a full year since then (the company I 
> work for gave me assignments other than DEM), I feel that perhaps I should 
> reach out for other ways to share this information…
> 
> This implies that although I am quite confident of my fix, I am not 100% 
> certain either…
> 
> Perhaps a first step would be for me to present what I have found, and how my 
> fix works? I am not sure as to where to present it: This mailing list? Thread 
> #694548? You tell me 😉️
> 
> Gaël
> 
> 
> PS: I have started to re-implement my solution in yade e07e530 (26 Nov 2020) 
> (my previous implementation was based on yade 1b4ae97 (8 Feb 2020)). 
> Hopefully the work I do on e07e530 will not be difficult to port to the 
> latest yade versions… (It seems the yade project started using clang-format 
> (or similar tool) sometime between e07e530 and 1b4ae97 so `diff` outputs half 
> the file: not very helpful… 😆️)
> 
> PS2: Could thread #695558 be also concerned by this same uncorrect physical 
> behavior?
> 
> PS3: I hope that I am not colliding with Bruno's work on the subject 
> (according to Bruno's e-mail of Tue, 12 Jan 2021 07:50:51)
> 


-- 
--
Janek Kozicki, PhD. DSc. Arch. Assoc. Prof.
Gdańsk University of Technology
Faculty of Applied Physics and Mathematics
Department of Theoretical Physics and Quantum Information
--
http://yade-dem.org/
http://pg.edu.pl/jkozicki (click English flag on top right)

_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to     : yade-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yade-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to