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

[Column List][Bom][Added] Added 'Net Label' column #731

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

nguyen-v
Copy link

This PR adds a new column Net Label, which is similar to to Net Name but without the path. Only displays the Net Label given in the schematic. For example, for the nets:

/Project Architecture/MCU - IOs/MOTOR_ENABLE
/Project Architecture/MCU - IOs/MOTOR_HIZ
/Project Architecture/MCU - IOs/MOTOR_FAULT
/Project Architecture/MCU - IOs/DRV_SCLK
/Project Architecture/MCU - IOs/DRV_MISO

Net label would be:

MOTOR_ENABLE
MOTOR_HIZ
MOTOR_FAULT
DRV_SCLK
DRV_MISO

If there is no '/', just keep it as is, for example:
+3V3 would stay +3V3

This is useful when creating simplified testpoint lists in a large hierarchical schematic, where the path can get very long.

Similar to Net Name but without the path. Only displays the Net Label given in the schematic
@nguyen-v
Copy link
Author

Not sure how to handle the 'Invalid column name' warning, I don't know if this new column should be added to the extra columns as it is inherently redundant with Net Name

@set-soft set-soft merged commit 053e54f into INTI-CMNB:dev Nov 22, 2024
17 checks passed
@set-soft
Copy link
Member

Thanks!
I added it to the "extra" columns, they are used to detect bogus names and for examples. A redundant column in an example isn't bad.

@nguyen-v nguyen-v deleted the feat_col_net_label branch November 22, 2024 10:08
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.

2 participants