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

creating tool_db_utils #2654

Closed
wants to merge 5 commits into from
Closed

Conversation

spacestate1
Copy link

I've added a c program to the sim/axis/db_demo to pull in tools form the sqlite3 databse. it's mostly based on the python code in the db_demo. It's basic But a good start to get Linuxnc working with sqlite3 and might open the door to track things like tool life and other things.

@rene-dev
Copy link
Member

I have a branch which integrates sqlite as a tooltable directly. It finally and once and for all solves all tooltable related issues properly. I will clean it up and create a PR.

@spacestate1
Copy link
Author

That would be awesome. Does it integrate the same way using the emc/tooldata code ?

@andypugh
Copy link
Collaborator

Do either of these use the tool database schema that I worked out with Tormach?

http://wiki.linuxcnc.org/cgi-bin/wiki.pl?ToolDatabase

@spacestate1
Copy link
Author

spacestate1 commented Sep 23, 2023

My code does not use that, but it can. I used a simple test db to make sure the connections were working and loading tools. I see spindle hrs in the db, and one of my main motivations is to track individual tool usage time for tool life etc. It looks like that can be tracked per tool in this db ?

@spacestate1
Copy link
Author

@andypugh I've re-written the code to incorporate the database you mentioned here:
http://wiki.linuxcnc.org/cgi-bin/wiki.pl?ToolDatabase

It works the same it did before, but now it loads tools from that database into LinuxCNC. Now other features can be added to start tracking things like tool life and spindle usage.

Here's the relevant changes
https://github.com/spacestate1/linuxcnc/tree/master/configs/sim/axis/db_demo/tool_db_utils

@rene-dev
Copy link
Member

my code depends on this PR: #2497
It greatly simplifies tool handling, and fixes all root causes of the problems with the tooltable.
It can adapt to any schema.

@spacestate1
Copy link
Author

I'll take a look. The code in this pull just interacts with the current system to load/unload tools and doesn't change anything internally.

@gmoccapy
Copy link
Collaborator

gmoccapy commented Oct 7, 2023

stuttgart meeting agrees this will be obsolete when @rene-dev is done with the sqlite tooltable.

@gmoccapy gmoccapy closed this Oct 7, 2023
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.

4 participants