Skip to content
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

ira_eq.x doesn't work. #1

Closed
FanMover opened this issue Nov 15, 2021 · 1 comment
Closed

ira_eq.x doesn't work. #1

FanMover opened this issue Nov 15, 2021 · 1 comment

Comments

@FanMover
Copy link

Hi , @mgoonde .

I am a student working on a project of Off-lattice Kinetic Monte Carlo. I found your paper and source code with excitement. As you mentioned in the paper(IRA: A shape matching approach for recognition and comparison of generic atomic patterns), in self-Learning KMC, atomic assignment is unknown amd number of atoms unset. Traditional optimal method like SVD and assignment method like Hungarian algorithm cannot work on my system. So I downloaded your code and tried.

However, there was something wrong after I combined two .xyz system and executed ira_eq.x.

First, variable hd_fin not defined. To be honest, I don't know what it means. So i changed
hd_out = hd_fin
to
hd_out = hd
image

Second, the way you permute the coordinates is not recognized by my machine, it goes
image
, whenerver there is a permutation operation. So I changed it from
coords1(:,:) = coords1(:,nint(d_o(2,:)))
to
do i = 1, nat1 coords1(:,i) = coords1_tmp(:,nint(d_o(2,i))) end do
So does the array typ1/typ2/coords2

After doing these rescue measures, I ran your example ./ira_eq.x < temp_ira. temp_ira is combined from benchmark_test/data/lj_clusters/xyz/47.xyz, same structure file in run_single.sh, and the randomized one. The two xyz structures didn't found scattered using ovito. The result goes infinity.

I don't know what is wrong and I am not that farmiliar with fortran. So I appreciate your help about it. Best wishes.

@mgoonde
Copy link
Collaborator

mgoonde commented Nov 15, 2021

Hey @FanMover,
I would be interested in your project, since we are actually doing the exact same thing (off-lattice kmc). We will have a release of the kmc quite soon, and it will be posted also in the description here. I would appreciate if you could write me a short email of what you do.

Regarding the code problem:

  • You are right, the variable hd_fin is not defined. I have pushed a commit fixing this.
  • it's strange that your machine does not recognize this way of permutations. Which compiler do you use? I will change it to a more general solution (exactly as you wrote), later today

mgoonde added a commit that referenced this issue Nov 15, 2021
hopefully more general for different compilers (see issue #1)
@mgoonde mgoonde closed this as completed Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants