-
Notifications
You must be signed in to change notification settings - Fork 8
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
Enable particle input from file #33
base: main
Are you sure you want to change the base?
Conversation
Updated here now that the crack branching problem is merged; also split into a new example to separate the specific file format from the source |
@streeve I tried to compile the new version of the example.
However, I get some Kokkos errors while compiling.
Could you please have a look? |
@diehlpk I fixed the compilation locally - should be much more usable now |
@streeve thanks, I will compile and test a uniform bar. |
I get now a new error which was not there before:
|
Sorry, I forgot to say you'll need to update to the latest Cabana master (readme has been updated with the minimum commit necessary) |
@streeve The code compiles now, but we get a segmentation fault after reading the CSV file. I nailed it down between CabanaPD/examples/read_file.cpp Line 43 in 8fb8f42
CabanaPD/examples/read_file.cpp Line 84 in 8fb8f42
Here is the output:
and with gdb, we get
I am attending a conference this week and just had a few minutes to test things. I can have a more detailed look next week. |
If you attach an example file I can test as well |
This is the CSV file easy.csv and the image of the corresponding mesh |
@diehlpk as you can see from the last commit, there were a lot of necessary changes. Runs for me now locally |
@streeve thanks. I will try to run a larger geometry and apply the boundary conditions. |
To do:
|
@diehlpk made the changes we discussed - let me know if you have any issues or if you have larger files for me to test with |
@streeve thanks, I will upload the coarsest grid, we used in the previous simulations this evening. |
@streeve Please find attached a mesh more suitable for the comparison with the experiment. Here is the file astm-d638.csv If that mesh works, we would be good to setup the experimet. |
@streeve mayve we should add a very small CSV file to the repo to execute the example. What do you think? |
issues with LinkedCellList (in VerletList) with single bin in a given dimension
I'm okay with that as long as it's very small (a dog-bone with 100s of particles would be nice) |
I ran with the astm file and can correctly create the BC we want, but I'm not sure if the issues I see locally are because of input magnitudes or a bug |
e51430a
to
ab6f3e5
Compare
@diehlpk should be ready to test. The last commit adds the linear ramp for BC (as well as the option to turn off the BC at some point in the simulation). I also had to remove some duplicate positions with the test mesh I was using |
3978b6c
to
89c9e6c
Compare
@diehlpk let me know if you notice any issues - I can try to fix before we meet next |
@streeve I was attending SC. I will testbit before the meeting back home. |
@streeve I prepared a 3D mesh and uploaded the mesh to the shared folder. I will try this week to setup the 2d case with the experimental data. Could you please try the 3d mesh? |
Attempt to read particles from a CSV file.
Closes #9