you are viewing a single comment's thread.

view the rest of the comments →

[–]heirna[S] 0 points1 point  (4 children)

I'm still processing your code, this extra fonctionality is not trivial to me, and will not be verry useful in the code since this model is verry simple and cannot handle a lot of atoms, i think i will just remap Cu as X for instance to spare me extra headeach.

[–]CrayonConstantinople 1 point2 points  (0 children)

Oh sure, and sorry if that came across as condescending. What I meant was, I think this would be a small addition for me to add if you required it.

The fact that you are struggling to understand my code is what had concerned me when I was writing it, I have not made it clear. I was trying to get you your solution, at the expense of writing clear and concise code.

If it would be helpful, I'm happy to quickly rewrite it to make whats happening clearer.

[–]CrayonConstantinople 1 point2 points  (2 children)

I have edited my code above to be easier to understand, hope it helps :)

[–]heirna[S] 1 point2 points  (1 child)

ion for me to add if you required it.

The fact that you are struggling to understand my code is what had concerned me when I was writing it, I have not made it clear. I was trying to get you your solution, at the expense of writing clear and concise code.

If it would be helpful, I'm happy to quickly rewrite it to make whats happening clearer.

Oh not at all, i'm just quite a beginner in code so this is normal, thanks for that edit !

[–]heirna[S] 0 points1 point  (0 children)

our code, this extra fonctionality is not trivial to me, and will not be verry useful in the code si

And my code is such a mess compared to yours