Geo/update the readme for the c phi test#14490
Conversation
markelov208
left a comment
markelov208
left a comment
There was a problem hiding this comment.
Hi Sabine, thank you very much for cleaning this test and fixing the description. Follow your changes, I see that the original text is not really English. Could you also improve the text a bit? Thank you.
WPK4FEM
left a comment
WPK4FEM
left a comment
There was a problem hiding this comment.
Hi Sabine,
Glad to see that this also reduces the number of json files that is excluded from formatting. I have some small nagging about textual things, but am generally happy with the changes.
Regards, Wijtze Pieter
markelov208
left a comment
markelov208
left a comment
There was a problem hiding this comment.
Hi Sabine, thank you very much for processing the comments. I have two minor comments on the text.
markelov208
left a comment
markelov208
left a comment
There was a problem hiding this comment.
Hi Sabine, I think the PR can be merged. Let's wait for Wijtze Pieter's decision.
WPK4FEM
left a comment
WPK4FEM
left a comment
There was a problem hiding this comment.
Hi Sabine,
Although I think that we generally speak about gravity i.s.o. gravitation, the description is correct.
Good to go for me.
Regards, Wijtze Pieter
3 participants
Updated the README.md file for the C-Phi test case to point to the website and K0-procedure documentation.
Also formatted and cleaned the material and project parameter files.