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

Add a way to skip parameters in categories #54

Closed
randrej opened this issue May 18, 2024 · 7 comments
Closed

Add a way to skip parameters in categories #54

randrej opened this issue May 18, 2024 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@randrej
Copy link
Contributor

randrej commented May 18, 2024

Using your config, let's look at Operational Amplifiers. They're a subgroup of Amplifiers, which is a subgroup of Integrated Circuits. Since Amplifiers specify Supply Current, whenever you're importing an opamp (say, a TL072), you'll be asked for Supply Current which is not really something listed in opamp datasheets.

It'd be nice if there was an _omit_parameters field to ignore these things.

@randrej
Copy link
Contributor Author

randrej commented May 18, 2024

Btw I'd like to help on things like this, I'm not just asking for you to do something, but you gotta approve it first.

@30350n 30350n self-assigned this May 19, 2024
@30350n 30350n added the enhancement New feature or request label May 19, 2024
@30350n
Copy link
Owner

30350n commented May 19, 2024

The categories and parameters in my config are mainly based on DigiKey's structure. And regarding that specific example, DigiKey indeed specifies Supply Current (without Load) for Opamps. For example for the TL072HIDR. The value seems to come from the graphs on Page 28/29 of the datasheet.

Anyways, regarding the feature suggestion: I have actually come across a case in my configuration where this could have been useful, but the format generally works without it. You'll just have to define the parameters further down the category tree.

I'm not per se against this feature, but I fear that it might be pretty confusing.

@30350n
Copy link
Owner

30350n commented May 29, 2024

After thinking about this a little more, I guess it wouldn't really hurt much to implement this and it might makes configuration easier in some cases.

Regarding the naming, _omit_parameters is already not bad. What do you think about something like _omit_parent_parameters or _ignored_parent_parameters (mainly to convey that this affects parameters inherited from parent categories)?

@30350n
Copy link
Owner

30350n commented May 29, 2024

(Oh also, offtopic: Did you receive my e-mail reply btw? I had some issues with the DNS settings for my mail server the first time I sent it ^^)

@randrej
Copy link
Contributor Author

randrej commented May 31, 2024

Hey, I got your e-mail!

Yeah, I agree with this, hopefully I'll find time soon to make an MR.

Also off topic here:

I'm also considering ways of incorporating the template part / variant functionality to have generic parts. I'll write an issue about it when I investigate it a bit further.

I'm currently doing an important project that uses your tool + Inventree, I want to see it through before suggesting any new functionality; I need to get a feel for how it works in real use.

@30350n
Copy link
Owner

30350n commented Jun 1, 2024

Yeah, I agree with this, hopefully I'll find time soon to make an MR.

I might also be able to give it a shot if I find some time. I think this should require pretty minimal changes, like 10 lines or so at max.

I'm also considering ways of incorporating the template part / variant functionality to have generic parts. I'll write an issue about it when I investigate it a bit further.

Definitely check out #20 (comment) if you do. That's the kind of functionality that I'd think makes the most sense here (probably with slightly different naming though).

I'm currently doing an important project that uses your tool + Inventree, I want to see it through before suggesting any new functionality; I need to get a feel for how it works in real use.

Cool! Let me know if you run into any issues.

@30350n
Copy link
Owner

30350n commented Jun 29, 2024

This has been implemented in b28047d!

@30350n 30350n closed this as completed Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants