Skip to content

Add test_2d_milling_2d_dat_cutter_planar example - #1043

Open
crb116923 wants to merge 11 commits into
Xiangyu-Hu:masterfrom
crb116923:test_2D
Open

Add test_2d_milling_2d_dat_cutter_planar example#1043
crb116923 wants to merge 11 commits into
Xiangyu-Hu:masterfrom
crb116923:test_2D

Conversation

@crb116923

Copy link
Copy Markdown

Professor Hu,
I made a mistake with the upload path before. What was the result this time? Thank you for your guidance.

@crb116923 crb116923 changed the title Add files via upload Add test_2d_milling_2d_dat_cutter_planar example Mar 21, 2026
Hello Professor Hu, 
I have corrected the error that occurred during the gcc build process. Thank you for your time.
@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 Now, as your case can be compiled. We can have discussion based on the new case. Could the case run?

@crb116923

crb116923 commented Mar 27, 2026

Copy link
Copy Markdown
Author

1.0023.bmp
 Professor Hu,

First of all, this case study can run. The problem is that only it runs partly, and I don't know why. Thank you for your time and guidance.

I need you detail the issue so that we can discuss.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923
I need you detail the issue so that we can discuss.

@crb116923

Copy link
Copy Markdown
Author

2026_04_03_report.pdf
Hello Professor Hu,
This week I did a presentation to explain my current progress. Thank you for your time and guidance.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 I have read your report. Did you have tried to tune some parameters, such as time step size (by cfl number) and dissipation factor of the solver?

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 I notice that you are using very hard material. did you try softer material for this problem?

@crb116923

Copy link
Copy Markdown
Author

Hello, Professor Hu,
I will run these operating conditions with different materials and different parameters.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 your cutter is not moving. please commit a valid setting up.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 your cutter is not moving. please commit a valid setting up.

Sorry. I found the place for motion.

@Xiangyu-Hu

Xiangyu-Hu commented Apr 7, 2026

Copy link
Copy Markdown
Owner

@crb116923 I have reproduced your results, there is a numerical instability. We can have a online discussion after you have collected the parameter study.

@crb116923

crb116923 commented Apr 8, 2026

Copy link
Copy Markdown
Author

Professor Hu,
Thanks for your guidance.Do you have time this Friday at 2 PM German time?

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

German

I will be free on Friday 3pm.

@crb116923

Copy link
Copy Markdown
Author

Hello Professor Hu,

Thank you for your time.

曹庭赫 邀请您参加腾讯会议
会议主题:2026_04_10_Meeting
会议时间:2026/04/10 15:00-15:30 (GMT+02:00) 中欧时间 - 柏林

点击链接入会,或添加至会议列表:
https://meeting.tencent.com/dm/oYcFADCkaWcT

#腾讯会议:847-365-077

复制该信息,打开手机腾讯会议即可参与

auto &plate_update_pos =
main_methods.addStateDynamics<fluid_dynamics::UpdateParticlePosition>(plate);

auto &plate_correction =

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the bug is that we have forget the parameter for free surface correction parameter. Use about 0.5 will solve the numerical instabilty.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu, here is the result.

First, the parameters used in the case are a rotation speed of 12 revolutions per minute and jelly; this rotation speed is far below the real operating conditions. Three constraints were set. No debris was generated.

Second, if replaced with an aluminum block, it cannot run under the same conditions.

test_1.mp4

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu, here is the result.

First, the parameters used in the case are a rotation speed of 12 revolutions per minute and jelly; this rotation speed is far below the real operating conditions. Three constraints were set. No debris was generated.

Second, if replaced with an aluminum block, it cannot run under the same conditions.

test_1.mp4

Could you detail the cases which is you said "can not run"?

@crb116923

Copy link
Copy Markdown
Author

Second, if replaced with an aluminum block, it cannot run under the same conditions.

test_2.mp4
image image image

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

could you identify the material property range which can run.

@crb116923

Copy link
Copy Markdown
Author

Analysis.pdf
Professor Hu, these are the results of my attempts with nine types of plastic materials.

@Xiangyu-Hu

Xiangyu-Hu commented Apr 29, 2026

Copy link
Copy Markdown
Owner

Analysis.pdf Professor Hu, these are the results of my attempts with nine types of plastic materials.

@crb116923 please rescale your dimension system so that there is no absurd value like 2.7 m displacement for a drilling case.
Also, all values should be resacled so that the scaled values are between 0.001 to 1000. Or even narrower range.

@crb116923

Copy link
Copy Markdown
Author

Analysis.pdf
Professor Hu, I have modified the parameters and made the jelly's running time the standard.

@crb116923

crb116923 commented May 9, 2026

Copy link
Copy Markdown
Author

Hello Professor Hu, I didn't bother you last Friday because of the Labor Day holiday. 2.7m represents the case with two-sided constraints; 0.036m represents the case with three-sided constraints. If you need verification videos for all nine cases, I can provide them.

@crb116923

Copy link
Copy Markdown
Author

Analysis.pdf
Professor Hu,

First, I'll submit a version based on this.

Second, I'd like to ask you, does scaling to 0.001 to 1000 include normal parameters like density and mass?

Third, the pressure of studying and changing careers is exhausting me. As I mentioned before, please feel free to ask me any questions; I'll continue learning.

Updated material properties and simulation parameters for the 2D milling test. Added bottom fixed region and adjusted velocities and time settings.
@crb116923
crb116923 requested a review from Xiangyu-Hu May 9, 2026 17:09
@crb116923

crb116923 commented May 12, 2026

Copy link
Copy Markdown
Author

An idea
An idea.docx

This idea is wrong because there is gunpowder.

@crb116923

Copy link
Copy Markdown
Author

Dear Professor Hu, I have a new idea to share with you.
elasto-plastic contact  milling mechanical simulation
Overall.docx

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 I think that is too much for a master thesis, I think that you may just need a make the present model can generate meaningful results.

@crb116923

Copy link
Copy Markdown
Author

ok,I am modifying the code now.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 actually, you can work on your project based on SPHinXsim repository, there is a milling case works already.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu, I noticed that the material currently used for the Taylor Bar case is Hardening Plastic Solid, while my case uses J2 material. Second, I tried modifying the code API, but it never worked.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu, I noticed that the material currently used for the Taylor Bar case is Hardening Plastic Solid, while my case uses J2 material. Second, I tried modifying the code API, but it never worked.

what do you mean never worked. Please specify the issue so that we can discuss it. Words like never worked have no help at all.

@crb116923

Copy link
Copy Markdown
Author
image I modified the code by cloning the case based on the Taylor version and re-uploaded the relevant files. The code now builds successfully, but it still reports an error when running the case. Therefore, although the compilation succeeds, the case cannot actually run. In other words, the code builds, but it still does not work at runtime.

I understand that this may not be a simple compilation error. It seems more likely to be a runtime issue related to SYCL/kernel/device-side execution, or possibly caused by some inconsistency in the case configuration, kernel registration/linking, or runtime environment. The runtime error message is shown below.

How do you see this issue? Which parts would you suggest I check first?

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

image I modified the code by cloning the case based on the Taylor version and re-uploaded the relevant files. The code now builds successfully, but it still reports an error when running the case. Therefore, although the compilation succeeds, the case cannot actually run. In other words, the code builds, but it still does not work at runtime.
I understand that this may not be a simple compilation error. It seems more likely to be a runtime issue related to SYCL/kernel/device-side execution, or possibly caused by some inconsistency in the case configuration, kernel registration/linking, or runtime environment. The runtime error message is shown below.

How do you see this issue? Which parts would you suggest I check first?

You need identify which line of code leads to the crash.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu, first, I can run the relaxation successfully on the CPU using par_host, but it fails on the GPU using par_ck.

Second, I am not sure how to correctly make the corresponding kernel included in the SYCL device binary.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu, first, I can run the relaxation successfully on the CPU using par_host, but it fails on the GPU using par_ck.

Second, I am not sure how to correctly make the corresponding kernel included in the SYCL device binary.

But the code you have referred runs on GPU without issue. So you need check where is the difference you have changed leads to the issue.

@crb116923

crb116923 commented Jul 11, 2026

Copy link
Copy Markdown
Author

Professor Hu, first of all, I did not modify the code in the relaxation stage.

Second, the version currently installed on my computer is the latest library cloned from:

git clone https://github.com/Xiangyu-Hu/SPHinXsys.git SPHinXsysSYCL

Third, with the latest version, my case can be built successfully, but it cannot run on the GPU. I feel a bit frustrated because I did not change the relaxation-stage code, but this runtime issue still appears there.

Do you mean a test case from the repo is not able run on GPU. It is bit strange as they are all sycl cases have tested with in github atveach pull request.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu,

First, I have already run a considerable amount of simulation data before. Would it be acceptable for me to start writing a first draft of the paper based on the existing results?

Second, the latest version of my code currently has a runtime issue. Specifically, the GPU kernel path for UpdateCellLinkedList<RealBody> under par_ck fails. I am actively trying to identify the cause and modify the code.

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu, first of all, I did not modify the code in the relaxation stage.

Second, the version currently installed on my computer is the latest library cloned from:

git clone https://github.com/Xiangyu-Hu/SPHinXsys.git SPHinXsysSYCL

Third, with the latest version, my case can be built successfully, but it cannot run on the GPU. I feel a bit frustrated because I did not change the relaxation-stage code, but this runtime issue still appears there.

Do you mean a test case from the repo is not able run on GPU. It is bit strange as they are all sycl cases have tested with in github atveach pull request.

Do you mean a test case from the repository not able to run on GPU or your code not?

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu,

First, I have already run a considerable amount of simulation data before. Would it be acceptable for me to start writing a first draft of the paper based on the existing results?

Second, the latest version of my code currently has a runtime issue. Specifically, the GPU kernel path for UpdateCellLinkedList<RealBody> under par_ck fails. I am actively trying to identify the cause and modify the code.

Sure. If you think in that way.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu, first of all, I did not modify the code in the relaxation stage.
Second, the version currently installed on my computer is the latest library cloned from:
git clone https://github.com/Xiangyu-Hu/SPHinXsys.git SPHinXsysSYCL
Third, with the latest version, my case can be built successfully, but it cannot run on the GPU. I feel a bit frustrated because I did not change the relaxation-stage code, but this runtime issue still appears there.
Do you mean a test case from the repo is not able run on GPU. It is bit strange as they are all sycl cases have tested with in github atveach pull request.

Do you mean a test case from the repository not able to run on GPU or your code not?

yes,this test case from my repository can build but not run on GPU.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu,
First, I have already run a considerable amount of simulation data before. Would it be acceptable for me to start writing a first draft of the paper based on the existing results?
Second, the latest version of my code currently has a runtime issue. Specifically, the GPU kernel path for UpdateCellLinkedList<RealBody> under par_ck fails. I am actively trying to identify the cause and modify the code.

Sure. If you think in that way.

Thank for your reply.I will try to write immediately

@crb116923

Copy link
Copy Markdown
Author

MA_CRB.pdf
Hello Professor Hu,

This is the first draft of my paper.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu,

First, I have now reinstalled the NVIDIA GPU driver, and the problem has been resolved.

Second,When would you be available to meet in person between August 4 and August 11?

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

Professor Hu,

First, I have now reinstalled the NVIDIA GPU driver, and the problem has been resolved.

Second,When would you be available to meet in person between August 4 and August 11?

Basically, I am free on Friday afternoons. we can meet in person then.

@crb116923

Copy link
Copy Markdown
Author

Professor Hu, this week I followed the Taylor bar example and changed the material model from J2 to HardeningPlasticSolid. I tested three materials. The mild steel block and aluminum simulations could only run partially, while the jelly showed the rotational behavior below.

jelly.mp4

@Xiangyu-Hu

Copy link
Copy Markdown
Owner

@crb116923 please come to university on Friday afternoon so that we can have in person discussion.

@crb116923

Copy link
Copy Markdown
Author

@crb116923 please come to university on Friday afternoon so that we can have in person discussion.

Professor Hu, see you on August 7th. Thank you for your time

@crb116923

Copy link
Copy Markdown
Author

Dear Prof. Hu,

I have some good news. The code can now be successfully built and runs correctly on Linux.

I have tested two constitutive models (J2 and HardeningPlasticSolid) together with both the fluid integration algorithm and the solid integration algorithm. Based on the current numerical results, the fluid integration algorithm provides more stable and physically reasonable results for the present milling case, while the solid integration algorithm still exhibits some numerical issues for certain materials. I am currently investigating the underlying reasons.

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

Successfully merging this pull request may close these issues.

2 participants