FreeCAD_Assembly4 icon indicating copy to clipboard operation
FreeCAD_Assembly4 copied to clipboard

BOM-Expansion

Open Ediforce44 opened this issue 3 years ago • 8 comments
trafficstars

Hey, I didn't find a better way to contact someone from this repository :D. So I hope this is the way to go.

I created a little expansion for the BOM mechanism (here [1]). I want to ask if you are interested in integrating this directly into Assembly 4. If so I would help you to make the integration cleaner than it is know, because not everybody needs these features and a option menu would be great.

Story time: I saw Assembly 2 or so featuring a BOM mechanism which is useful for woodworkers and everyone how build something out of plane parts. So I automized the BOM of A4 to do the same and even more and better for the above usecase ;). I hope you like it.

Ediforce44 avatar Oct 13 '22 07:10 Ediforce44

short answer: yes, I'm interested. Please note that there have been quite some improvements and additions in Asm4 BOM recently.

Thinking of various BOM needs and functions, I'd suggest to have a drop-down menu in the toolbar, similar to the fasteners or datum objects menu in the menubar, which would allow to call the chosen BOM function. In this case, "your" BOM would be one of the available options.

Do you think you could make a proposal for such a thing ? (in any case, please make PRs against the development branch and not master)

Zolko-123 avatar Oct 13 '22 07:10 Zolko-123

Sure. I am new to Pull Requests. Should I create a new PR with base: master and compare: development, or what should I do.

Ediforce44 avatar Oct 13 '22 08:10 Ediforce44

Have you tried the developer branch? There is this BOM functionality working there.

leoheck avatar Oct 15 '22 15:10 leoheck

I came here with another idea to implement in the existing BOM. Which is a flag to disable internal parts to be added in the BOM. For instance, I assembled a Motor with Asm4, but I don't want parts of the motor in the BOM, because the motor is a single part since I am not making motors. This thing is missing and I am going to add it to the existing BOM system.

leoheck avatar Oct 15 '22 15:10 leoheck

Nice, I implemented something similar in my version. If the part, model etc. is invisible it would not be included into the BOM. But I think the need for tags is something different since it is permanent and independent from the rendering.

Ediforce44 avatar Oct 15 '22 18:10 Ediforce44

Btw I saw you worked a lot on the BOM. So I ask you what do you think about that: I think about a selection menu to select your BOM method for the project. Every BOM method has its own starting function. If you create the BOM by pressing the button the corresponding starting function is called. The starting function will then create a partlist with attributes and the BOM will process this list after the starting function is finished. (Like a strategy design pattern if you know what I mean).

So it would be really easy to add new BOM methods

Ediforce44 avatar Oct 15 '22 18:10 Ediforce44

What if you make a pr proposing something so we can visualize better?

leoheck avatar Oct 15 '22 20:10 leoheck

I will do it as soon as I have time for it.

Ediforce44 avatar Oct 15 '22 20:10 Ediforce44

Hello, I don't know who worked on the BOM, but it was making errors, so I cleaned up a bit (in the development branch). A lot of stuff is still hard-coded (Doc_Label), please use the InfoKeys tables instead.

Zolko-123 avatar Nov 14 '22 00:11 Zolko-123

closed, see #395

Zolko-123 avatar Feb 09 '23 16:02 Zolko-123

I will do it as soon as I have time for it.

Hi @Ediforce44,

Can we help in testing or some how... I am reading on the issue, and your proposal of having multiples BOM methods come to mind as soon as I stumble on Zolko noticing that the BOM feature perish over time because of divergent needs... That way we could rely on a working BOM feature.

I saw your video, but I am not sure if applying your mod over current plugin version would work as Zolko mention he makes some clean up...

@Zolko-123, if @leoheck implementation works can we merge it? Some thing better than no thing...

:)

BuhtigithuB avatar Feb 20 '23 03:02 BuhtigithuB

Sadly I have to say, I couldn't do anything. I have a lot going on in real life and also a lot of exams. I couldn't really follow the newest changes, so I don't know if I could help in any way. If leoheck has a working version I think it is better to go with his one.

Ediforce44 avatar Feb 20 '23 11:02 Ediforce44

I put the old, existing BOM to work. It was hard and it may have issues, of course, but it works listing important objects. There might be some objects that are missing because each one has to be treated individually. Also, the code can be improved, always. But this is a matter of putting extra time into having this merged in the master to allow people to bring their issues so we can build it incrementally. Having this feature lost in the development branch does not help to improve this because people use to install add-ons with the addon manager. Now the addon manager could have the option to select the branch of the addon. I don't think it has. This could help users to test things easily.

leoheck avatar Feb 20 '23 13:02 leoheck