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.

Sign up for GitHub

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

Jump to bottom

Add new argument to high_level_function decorator that specifies dependencies #2953

Open
agoose77 opened this issue Jan 17, 2024 · 1 comment · May be fixed by #2967
Open

Add new argument to high_level_function decorator that specifies dependencies #2953

agoose77 opened this issue Jan 17, 2024 · 1 comment · May be fixed by #2967
Assignees
@agoose77
Labels
feature New feature or request
Projects
Finalization

Comments

@agoose77
Copy link
Collaborator

Description of new feature

  • Add declarative dependency specification at high_level_function
    • Declare versions & dependency group
    • Annotate docstring with this information
    • Check package versions once
@agoose77 agoose77 added the feature New feature or request label Jan 17, 2024
@agoose77 agoose77 self-assigned this Jan 17, 2024
@jpivarski
Copy link
Member

For record-keeping, this is addressing #2952. (A PR that fixes it should close both issues.)

@agoose77 agoose77 linked a pull request Jan 19, 2024 that will close this issue
Open
@jpivarski jpivarski added this to Unprioritized in Finalization Jan 19, 2024
@jpivarski jpivarski removed this from Unprioritized in Finalization Jan 19, 2024
@jpivarski jpivarski added this to Set aside (don't do) in Finalization May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agoose77 agoose77

Labels
feature New feature or request
Projects
Finalization
Set aside (don't do)
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: decorator to validate and annotate dependencies scikit-hep/awkward
2 participants
@agoose77 @jpivarski