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

Tests and changes for decorators #102

Merged
merged 19 commits into from
Apr 30, 2024
Merged

Tests and changes for decorators #102

merged 19 commits into from
Apr 30, 2024

Conversation

khatchad
Copy link
Member

@khatchad khatchad commented Apr 25, 2024 •
edited
Loading

  • Decorator testing.
  • Actually capture the decorated function for @tf.function decorated functions.
  • Add pytest.mark.skip summary.
  • Use pytest summary for all Python analysis. There are some decorators it may work for.
    • Test pytest decorator.

@khatchad
Add tests.
1c69bdb
@khatchad
More tests.
69e38fa
@khatchad
New test.
1db59e7
@khatchad
Format.
21d450c
@khatchad
Fix one test.
1c88d86
@khatchad
Update test.
3fee504 
If the decorator isn't defined, then let's not track it.
@khatchad
Remove tf.function parameters.
d683737 
Now that the decorators have been seemingly fixed, this summary is now
causing a failure.
@khatchad
Accommodate test case.
099f95d 
In light of wala#188.
@khatchad
Modify another test case.
f8c4916 
In light of wala#188.
@khatchad
Format.
644c562
@khatchad
Progress.
94612df
@khatchad
Add decorator test.
a1c1d0c
@khatchad
Put the old annotation code back.
2a6e522 
No sense in just swapping problems.
@khatchad
"Workaround" tests.
4ace23a
@khatchad
Merge branch 'master' into custom_decorators
b951c6e
@khatchad
Merge branch 'master' into custom_decorators
2e1fd0c
@khatchad
Merge branch 'master' into custom_decorators
638794f
@khatchad
Merge branch 'master' into custom_decorators
6d783e9
@khatchad
Add Pytest stuff.
66da155 
Why not? It may work on some decorators.
@khatchad khatchad marked this pull request as ready for review April 30, 2024 18:29
@khatchad khatchad changed the title Support custom decorators Some changes for decorators Apr 30, 2024
@khatchad khatchad changed the title Some changes for decorators Test and changes for decorators Apr 30, 2024
@khatchad khatchad changed the title Test and changes for decorators Tests and changes for decorators Apr 30, 2024
@khatchad khatchad enabled auto-merge (squash) April 30, 2024 18:33
@khatchad khatchad merged commit 43f337c into master Apr 30, 2024
3 checks passed
@khatchad khatchad deleted the custom_decorators branch April 30, 2024 18:36
khatchad added a commit that referenced this pull request May 2, 2024
@khatchad
Also ignore PyDev project metadata (#102)
f6bc883 
Minor modification to `.gitignore`.
khatchad added a commit that referenced this pull request May 2, 2024
@khatchad
Tests and changes for decorators (#102)
767b419 
- Decorator testing.
- Actually capture the decorated function for @tf.function decorated functions.
- Add `pytest.mark.skip` summary.
- Use pytest summary for all Python analysis. There are some decorators it may work for.
  - Test pytest decorator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@khatchad