Skip to content

Generators for ESP32 S3 and S3mini - #112

Open
bill-fitch4793 wants to merge 6 commits into
david284:mainfrom
bill-fitch4793:main
Open

bill-fitch4793 wants to merge 6 commits into
david284:mainfrom
bill-fitch4793:main

Conversation

@bill-fitch4793

Copy link
Copy Markdown

S3 can use 8 16 or 24 channels. S3 mini can use 8 channels. Only other board that will work is the Development board Version E which I have not yet tested.

edit generate_all.sh to process CANMIOESP generator

create new generators with CANMIO generator defaults
@bill-fitch4793

Copy link
Copy Markdown
Author

Development board Version E should use the same files

@david284

david284 commented Sep 8, 2026

Copy link
Copy Markdown
Owner

I notice that there are three MDF's with the id of '0D20' and the same version number
Yes, they have different 'names', but the idea of the ID's is that they uniquely describe the module, the name assigned to the ID is for human consumption
I would rather see different module ID's for these different MDF's

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see this file used anywhere. Was this file something you created first and then merged into generate_CANMIO.py ?

@david284

david284 commented Sep 8, 2026

Copy link
Copy Markdown
Owner

its the CANEPMIOxx trio of files, and I see then in the 'files changed' tab

{
"generated": "Generated by generators/generate_CANMIO.py -t CDU -p23 -v 4d",
"timestamp": "202609011524",
"generated": "Generated by ./generators/generate_CANMIO.py -t CDU -p23 -v 4d",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, this is not right. I guess you ran the generator as "sh ./generators/generate_all.sh" where I ran it as "sh generators/generate_all.sh".
Tiny, subtle difference. But it is annoying that all files are regenerated and get a new timestamp just because of this little difference.
Please revert these JSON files and run the generator with "sh generators/generate_all.sh" next time.

I'll add a todo item for myself to fix the generate_all.sh script to trim off the "./" from the path.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But I didn't run the generator - I just viewed the files in your push request....

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment was for Bill.

@SvenRosvall

SvenRosvall commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

I am thinking more about how we work with MDF's for own projects. One thing is to use the DEV manufacturer ID (13). Another is where to keep them, which implies how widely they are spread. When we started the ModuleDescriptor project we kept all MDF's we could come up with here. Later we realised that this is not the best idea. Instead we have moved to keep the MDF's for user projects within the application code repository. This is what we do for all the VLCB-Arduino example sketches now. However there are still a few example MDF's left here.

My own Arduino projects keep their MDF's with the sources. This goes for CANNX and CANLDR.

Keeping the MDF's with the application source makes it easier to keep the application and MDF in sync. I would actually argue for doing the same with existing MDF's for CBUS modules that exist in the kitlocker. But there are too many obstacles for this to be a reality.

Keeping DEV projects here means that the module ID must be unique, which defeats the purpose of the DEV manufacturerID.

So, I suggest that you move the MDF's for your ESP projects to your own repository. Yes, I admit this is contrary to what I wrote on the forum a while ago. Sorry about the confusion.

@david284

david284 commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Yes, I had very similar thoughts about what MDF's are published with MMC, during a session I was on with Thames Valley AG last night.
As we expect any developer to be able to use DEV ID (13) with any module ID, then I don't really want to create conflicts with MDF's built into MMC, so I'm inclined to remove any DEV module MDF's from the MMC deliverable

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.

3 participants