-
Notifications
You must be signed in to change notification settings - Fork 36
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
exclude variables annotated with ClassVar #75
Comments
It would be nice to have an annotation specifically to make Tap ignore variables, e.g., class ArgParser(Tap):
params: List[float] = []
""" Even number of entries. Each pair is the mean and
standard deviation of the Gaussian underlying a
log-normal distribution """
@tap_ignore
distributions: List[LogNormal] = []
""" process_args will construct this from params after
checking params for validity """ |
Hi @adrianmgg and @RadixSeven, We like both of these ideas! Thank you so much for bringing up this issue. We'll try to get to this soon, but any PRs are welcome and appreciated. Thanks, |
ClassVar
is (quoting from here in typing docs, emphasis mine):I think it would make sense to exclude any variables whose annotations are wrapped in
ClassVar
from the list of arguments.The text was updated successfully, but these errors were encountered: