Hi,
See https://github.com/FreeCAD/FreeCAD/pull/4007
Add open Nastran95 'inp' file function. It consists of 3 commits with FemMesh.cpp & FemMesh.h.
It is talked here: https://forum.freecadweb.org/viewtopic.php?f=18&t=51569
Later made another 2 commits with PythonConsole.cpp & PythonConsole.h.
It adds function to run a py file in python console. It is talked here: https://forum.freecadweb.org/viewtopic. ... 7&start=10
It was intended to another PR, but they joined together. Haven't figure out how to separate them.
Cheers,
John
PR #4009 FEM: Add readNastran95() function in FemMesh.cpp
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Be nice to others! Respect the FreeCAD code of conduct!
PR #4009 FEM: Add readNastran95() function in FemMesh.cpp
Last edited by johnwang on Sat Oct 31, 2020 9:58 pm, edited 1 time in total.
Re: PR #4007 FEM: Add readNastran95() function in FemMesh.cpp
I've separated them for you in to 2 different branches as I mentioned in the PR.
I've also taken the liberty to modify the code ever so slightly
I've also taken the liberty to modify the code ever so slightly
Alone you go faster. Together we go farther
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Re: PR #4007 FEM: Add readNastran95() function in FemMesh.cpp
Thank you very much.
Yeah, I should use branch.
It seems if I have two branches, I could raise PR from each branch. If I have no branch, all following commits will join the former ones in one PR? So I have to stop further working until the PR is solved.
Will this PR pull from your branches now, so I could delete my one and create new branch for new work?
Re: PR #4007 FEM: Add readNastran95() function in FemMesh.cpp
You're welcome.
...and you must use branch. That's how you collaborate using git.
Correct.
You didn't create a branch at all and therefore all your commits are being submitted in the PR.
Not necessary. This is where my git-fu is weak and limited so this might not be the best or ideal way to do it: What you can do is pull the specific branch I modified (that just addresses adding the functionality to open Nastran95 files) to your computer. 'Drop' (this is git-speak for deleting commits) your previous commits. So you end up seeing only the commits I've made. Then force pushing those changes to the PR you made.
I just realized you will have to close the PR and open a new one just for adding the functionality to open Nastran95 files. This is because you didn't create a branch.
Alone you go faster. Together we go farther
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Please mark thread [Solved]
Want to contribute back to FC? Checkout:
'good first issues' | Open TODOs and FIXMEs | How to Help FreeCAD | How to report Bugs
Re: PR #4009 FEM: Add readNastran95() function in FemMesh.cpp
PR 4007 is closed.
A new branch is created and new PR raised from there.
https://github.com/FreeCAD/FreeCAD/pull/4009
A new branch is created and new PR raised from there.
https://github.com/FreeCAD/FreeCAD/pull/4009