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

New Materials #58

Closed
wants to merge 9 commits into from
Closed

New Materials #58

wants to merge 9 commits into from

Conversation

DaviKaur
Copy link
Contributor

Added two new materials in list. Corrected a little spell mistake in Readme.txt file.

@PrzemoF
Copy link
Contributor

PrzemoF commented Oct 22, 2015

The "-a" commit is strange and probably invalid. The commit title is definitely invalid :-)
Please give those 2 "Glass Materials" different titles to describe what's inside. 2 of them should be merged together. I'm fairly sure we don't use brackets for units. That merge commit should not be in the pull request. No spaces in the field names, so "Compressive Strength" --> "CompressiveStrength". Density in kg/m^3 please. Price should be SpecificPrice. Please see src/Mod/Material/StandardMaterial/Tools/headers for supported headers. If you need more please discuss on the forum. (What is Softening T ?). Please get yourself familiar with src/Mod/Material/StandardMaterial/TEMPLATE.FCMat

@DaviKaur
Copy link
Contributor Author

Thank you so much for this nice and calm reply. I will correct all my mistakes.

@DaviKaur DaviKaur closed this Oct 28, 2015
WandererFan added a commit to WandererFan/FreeCAD that referenced this pull request Jul 12, 2016
WandererFan added a commit to WandererFan/FreeCAD that referenced this pull request Jul 12, 2016
bblacey pushed a commit to bblacey/FreeCAD-MacOS-CI that referenced this pull request Aug 21, 2016
PrzemoF referenced this pull request Dec 16, 2020
Path: Added missing signal-connection causing spindle speed not to be updated using ToolControllerDlg Bug 4479
PrzemoF pushed a commit to PrzemoF/FreeCAD that referenced this pull request Apr 6, 2021
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.

None yet

2 participants