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

R1,R2,R3 could become R1-R3 #20

Open
hildogjr opened this issue Jul 31, 2018 · 2 comments · May be fixed by #94
Open

R1,R2,R3 could become R1-R3 #20

hildogjr opened this issue Jul 31, 2018 · 2 comments · May be fixed by #94
Labels
wishlist Good to have features

Comments

@hildogjr
Copy link
Contributor

R1, R2, R3, R4, … could become R1-R4 on order_refs(foo) of the KiCost file have the function to do that.
https://github.com/hildogjr/KiCost/blob/master/kicost/eda_tools/eda_tools.py

I did implement on my pull request because I have to check if “R1-R4” will affect the search box.

I also added a example (acquire-PWM - iBOM.html.zip) to see why I think this is interesting.

@qu1ck
Copy link
Member

qu1ck commented Aug 1, 2018

Whoa, that's quite a complex board. Do you mind if I add it as one of demos? If you can share the pcb file, of course.

This will definitely affect search box. If this were to be done it should be done in javascript on the fly, not in python.

@qu1ck qu1ck added the wishlist Good to have features label Aug 1, 2018
@qu1ck qu1ck mentioned this issue Aug 1, 2018
@hildogjr
Copy link
Contributor Author

hildogjr commented Aug 2, 2018

True, but javascript is not my expertise at the moment.
You can add this HTML to the demos, just keep some reference to me at the file/folder/ README file about this, the kicad_pcb file that I am not able to share yet (part of my PhD study).
Also, appear more interesting just create a demo folder on this repository (not keeping one alone for the demo boards).

As you comment on other topic, trim or round the double precision could be nice in terms of file size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wishlist Good to have features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants